Marcel Holtmann wrote:
> Hi Jose,
>
>   
>> Here are a couple of patches that implement the following
>> commands in dbus-test:
>> GetRemoteClass
>> GetRemoteServiceHandles
>> GetRemoteServiceRecord
>>
>> This required modifications to dbus-sdp.c. The problem was
>> that the call to dbus_message_get_args in this file used a NULL
>> pointer for the error structure. This patch adds a pointer in
>> the same way that dbus-adapter.c calls dbus_messsage_get_args.
>> The end result is that both GetRemoteService* calls now seem
>> to work.
>>     
>
> I am not following that argument. If we don't care about the error code,
> then why should we fill one it. If this is a problem, then the problem
> is in the D-Bus low-level library or the Python bindings.
>
> Regards
>
> Marcel
>   
Hi Marcel,

This was my first hack with dbus. I haven't read the docs so
I'm not sure how it all works. I just noticed that the code in
dbus-adapter.c is working so put the same changes on dbus-sdp.c
and it worked.

I don't think the patch to dbus-sdp.c should be put in. Someone
with more dbus expertise should fix it. The patch to dbus-test
is simple and adds the new commands.

Sorry, that I don't have much time to help in this area. My focus
is to try to figure out how to get multiple SCO channels to work.

Jose