2006-06-06 14:35:07

by Frederic Danis

[permalink] [raw]

2006-06-06 16:52:58

by Claudio Takahasi

[permalink] [raw]
Subject: Re: [Bluez-devel] [PATCH] DBus GetRemoteName problem

Hi Frederic,

It's necessary consider some extra conditions:

1. DiscoverDevices running
2. DiscoverDevicesWithoutNameResolving running
3. non D-Bus inquiry request(eg: hcitool, kbluetoothd, ..)
4. After receive a connection complete event, a HCI remote name
request is sent to update the remote name

The GetRemoteName request can arrive under these conditions.

I already fixed this bug, but it needs more tests.

Claudio.

--
---------------------------------------------------------
Claudio Takahasi
Instituto Nokia de Tecnologia - INdT

On 6/6/06, Frederic Danis <[email protected]> wrote:
> Hello,
>
> When I use GetRemoteName of dbus-test program I receive a
> DiscoveryCompleted signal after the RemoteNameUpdated (or
> RemoteNameFailed) signal.
> You can find attached a patch to fix this.
>
> Hope this helps.
>
> Regards
>
> Fred
>
> --
> -----------------------------------------------
> It is not by improving the oil lamp that one invents the electric bulb!
> -----------------------------------------------
> Danis Frederic PalmSource Europe
> Software engineer
> Mail : mailto:[email protected]
> -----------------------------------------------
>
>
>
> --- dbus-adapter.c.orig 2006-06-06 11:35:32.000000000 +0200
> +++ dbus-adapter.c 2006-06-06 14:51:57.000000000 +0200
> @@ -1255,8 +1255,7 @@
> * Otherwise, send the HCI cmd to get the remote name
> */
> if (dbus_data->discover_state == STATE_IDLE) {
> - if (!disc_device_req_name(dbus_data))
> - dbus_data->discover_state = STATE_RESOLVING_NAMES;
> + disc_device_req_name(dbus_data);
> }
>
> return error_request_deferred(conn, msg);
>
>
>
>
> _______________________________________________
> Bluez-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/bluez-devel
>
>
>


_______________________________________________
Bluez-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/bluez-devel