Hi Forrest, On Wed, May 06, 2009, Forrest Zhao wrote: > We found this bug when developing the HFP plugin for telephonyd. > > --- > audio/headset.c | 3 +-- > 1 files changed, 1 insertions(+), 2 deletions(-) > > diff --git a/audio/headset.c b/audio/headset.c > index 9f6b736..9b9be17 100644 > --- a/audio/headset.c > +++ b/audio/headset.c > @@ -1304,8 +1304,7 @@ void headset_connect_cb(GIOChannel *chan, GError *err, gpointer user_data) > else > hs->auto_dc = FALSE; > > - if (server_is_enabled(&dev->src, HANDSFREE_SVCLASS_ID) && > - hs->hfp_handle != 0) > + if (server_is_enabled(&dev->src, HANDSFREE_SVCLASS_ID)) > hs->hfp_active = TRUE; > else > hs->hfp_active = FALSE; I don't see how that's a bug. hs->hfp_active should be true only when we are connected to HFP. server_is_enabled(src, HANDSFREE_SVCLASS_ID) is supposed to return true if HFP support is enabled in general. These are two separate things (we could be connected to HSP even though HFP support is enabled if the headset only supports HSP, i.e. hs->hfp_handle == 0 in this case). Johan -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
