On Thu, 2013-01-24 at 10:16 +0100, Mikel Astiz wrote: > From: Mikel Astiz <[email protected]> > > Make sure the reply to SetConfiguration() is sent before the internal > hook is fired. This is important because the hook could have side > effects including D-Bus interfactions (i.e. transport Acquire() being > called during module startup). > --- > src/modules/bluetooth/bluetooth-util.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/src/modules/bluetooth/bluetooth-util.c > b/src/modules/bluetooth/bluetooth-util.c > index 92254f2..1a14c9c 100644 > --- a/src/modules/bluetooth/bluetooth-util.c > +++ b/src/modules/bluetooth/bluetooth-util.c > @@ -1356,11 +1356,13 @@ static DBusMessage > *endpoint_set_configuration(DBusConnection *conn, DBusMessage > pa_log_debug("Transport %s profile %d available", t->path, t->profile); > > pa_assert_se(r = dbus_message_new_method_return(m)); > + pa_assert_se(dbus_connection_send(pa_dbus_connection_get(y->connection), > r, NULL)); > + dbus_message_unref(r); > > if (old_any_connected != pa_bluetooth_device_any_audio_connected(d)) > run_callback(d, false); > > - return r; > + return NULL;
The function returns now always NULL, so the return type should be changed to void. -- Tanu _______________________________________________ pulseaudio-discuss mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss
