On Thu, 2013-01-24 at 17:48 +0100, Mikel Astiz wrote: > Hi Tanu, > > On Thu, Jan 24, 2013 at 3:01 PM, Tanu Kaskinen <tanuk at iki.fi> wrote: > > On Thu, 2013-01-24 at 10:16 +0100, Mikel Astiz wrote: > >> From: Mikel Astiz <mikel.astiz at bmw-carit.de> > >> > >> 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. > > The function might return non-null in case of error, after fail2, but > I could change this if you prefer. Ah, I missed that. No need to change it, I'll apply the patch as is. -- Tanu