[xpcom] Fix signal emission checks to actually iterate through the

whole list instead of aborting on first miss (whoops). Original patch
by Charlie Lenahan.

diff --git a/xpcom-dbusservice/DBusService.cpp b/xpcom-dbusservice/DBusService.cpp
index 9213e8b523f8dfef727d65bb88013af768a7bedb..51fef8c009d7fb5b4a92e29ec25f349dd82420fe 100644 (file)
--- a/xpcom-dbusservice/DBusService.cpp
+++ b/xpcom-dbusservice/DBusService.cpp
@@ -360,14 +360,14 @@ DBusHandlerResult _signal_filter(DBusConnection *connection,
             if (!t.IsEmpty() && !t.Equals(dbus_message_get_sender(message)))
             {
                 BDBLOG(("    sender does not match\n"));
-                break;
+                continue;
             }
             signal->GetObjectPath(t);
             BDBLOG(("    object    : %s\n", PromiseFlatCString(t).get()));
             if (!t.IsEmpty() && !t.Equals(dbus_message_get_path(message)))
             {
                 BDBLOG(("    objectPath does not match\n"));
-                break;
+                continue;
             }
 
             /* do callback */