[pulseaudio-commits] src/pulsecore

David Henningsson diwic at kemper.freedesktop.org
Thu Jun 18 02:02:36 PDT 2015


 src/pulsecore/module.c |    4 ++++
 1 file changed, 4 insertions(+)

New commits:
commit 8f12ef901c183454eecce2faefd759f040e738fd
Author: David Henningsson <david.henningsson at canonical.com>
Date:   Thu Jun 11 14:00:08 2015 +0200

    module: Warn about modules loaded after unload
    
    We have crashes related to modules loaded after unload. This added
    warning can provide some information about what that module is,
    which in turn can help us solve the crashes, hopefully.
    
    BugLink: https://bugs.freedesktop.org/show_bug.cgi?id=90108
    Signed-off-by: David Henningsson <david.henningsson at canonical.com>

diff --git a/src/pulsecore/module.c b/src/pulsecore/module.c
index 8683cc1..1d4187e 100644
--- a/src/pulsecore/module.c
+++ b/src/pulsecore/module.c
@@ -313,7 +313,11 @@ void pa_module_unload_all(pa_core *c) {
     pa_xfree(indices);
 
     /* Just in case module unloading caused more modules to load */
+    PA_IDXSET_FOREACH(m, c->modules, state)
+        pa_log_warn("After module unload, module '%s' was still loaded!", m->name);
+    c->disallow_module_loading = 1;
     pa_idxset_remove_all(c->modules, (pa_free_cb_t) pa_module_free);
+    pa_assert(pa_idxset_isempty(c->modules));
 
     if (c->module_defer_unload_event) {
         c->mainloop->defer_free(c->module_defer_unload_event);



More information about the pulseaudio-commits mailing list