diff options
| author | Mikel Astiz <mikel.astiz@bmw-carit.de> | 2012-05-22 16:58:18 +0200 |
|---|---|---|
| committer | Marcel Holtmann <marcel@holtmann.org> | 2012-05-22 20:27:19 +0200 |
| commit | d41b654bd7cf0723c26f38db7aaee34dbe88bc54 (patch) | |
| tree | a48784e2e7f18d5f87f0e3192996fccf6f173f75 /gdbus | |
| parent | a911bed90486601deefb9585ba5f441e3e2ce02c (diff) | |
| download | connman-d41b654bd7cf0723c26f38db7aaee34dbe88bc54.tar.gz connman-d41b654bd7cf0723c26f38db7aaee34dbe88bc54.tar.bz2 connman-d41b654bd7cf0723c26f38db7aaee34dbe88bc54.zip | |
gdbus: Fix incorrectly discarded signals
Signals with no arguments were incorrectly filtered out due to the NULL
inequality check.
Diffstat (limited to 'gdbus')
| -rw-r--r-- | gdbus/object.c | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/gdbus/object.c b/gdbus/object.c index 2dd7c0e2..900e7aba 100644 --- a/gdbus/object.c +++ b/gdbus/object.c @@ -612,16 +612,12 @@ static gboolean check_signal(DBusConnection *conn, const char *path, for (signal = iface->signals; signal && signal->name; signal++) { if (!strcmp(signal->name, name)) { *args = signal->args; - break; + return TRUE; } } - if (*args == NULL) { - error("No signal named %s on interface %s", name, interface); - return FALSE; - } - - return TRUE; + error("No signal named %s on interface %s", name, interface); + return FALSE; } static dbus_bool_t emit_signal_valist(DBusConnection *conn, |