Hi Luiz, On Wed, Sep 08, 2010, Luiz Augusto von Dentz wrote: > From: Luiz Augusto von Dentz <luiz.dentz-von@xxxxxxxxx> > > Stream should only be closed if sep is unlocked otherwise there is a > possibility that sep reconfiguration is triggered although the sep is in > use by the user application. > --- > audio/a2dp.c | 2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > diff --git a/audio/a2dp.c b/audio/a2dp.c > index b062aed..ef0df17 100644 > --- a/audio/a2dp.c > +++ b/audio/a2dp.c > @@ -1393,6 +1393,8 @@ unsigned int a2dp_config(struct avdtp *session, struct a2dp_sep *sep, > } > > if (l != NULL) { > + if (a2dp_sep_get_lock(tmp)) > + goto failed; > setup->reconfigure = TRUE; > if (avdtp_close(session, tmp->stream, FALSE) < 0) { > error("avdtp_close failed"); Thanks for the patch! It's now in the upstream tree. 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