diff options
author | Tanu Kaskinen <tanuk@iki.fi> | 2016-05-28 18:54:04 +0300 |
---|---|---|
committer | Arun Raghavan <arun@arunraghavan.net> | 2016-05-29 10:34:26 +0530 |
commit | 9c7e4b8eb9139ba494c5e3877a7f585f0eec9c62 (patch) | |
tree | 77a12d1a5be20b893e8bd060190f11f1fc034c5a /m4 | |
parent | 1d5dfccbb2b72abca6989a1c85b92e3e6e23bded (diff) | |
download | pulseaudio-9c7e4b8eb9139ba494c5e3877a7f585f0eec9c62.tar.gz |
dbus: fix crash on LoadModule()
Commit ae415b07a07c9fe70714d01c91980edb25d966de ("dbus: Use hooks for
module new and removed events") changed the new module monitoring from
the asynchronous subscription system. Previously handle_load_module()
created the new pa_dbusiface_module object before we got
a notification of the loading of the module, but now we get the
notification already within the pa_module_load() call. That resulted
in a crash, because the module_new_cb() created the
pa_dbusiface_module object before pa_module_load() returned, and then
handle_load_module() would create another pa_dbusiface_module object
for the same module.
This patch removes the pa_dbusiface_module_new() call from
handle_load_module(). module_new_cb() is now responsible for all
pa_dbusiface_module object creations, except the ones that are created
during the initialization of module-dbus-protocol.
Signed-off-by: Arun Raghavan <arun@arunraghavan.net>
Diffstat (limited to 'm4')
0 files changed, 0 insertions, 0 deletions