diff options
| author | Daniel Wagner <daniel.wagner@bmw-carit.de> | 2011-01-14 16:14:20 +0100 |
|---|---|---|
| committer | Marcel Holtmann <marcel@holtmann.org> | 2011-01-19 17:26:23 +0100 |
| commit | d47616bd4aaf431e58a0fa4559786f279b768a57 (patch) | |
| tree | 4758db5af2570e7c224c14be023748cd66d0a809 /gdbus/object.c | |
| parent | 1839bdc0f70c0601ccc9aac66957288354153de3 (diff) | |
| download | obexd-d47616bd4aaf431e58a0fa4559786f279b768a57.tar.gz | |
gdbus: invaldate_parent_data: walk the whole path down
Assume there is only one object registerd at "/". If we add a new
object at "/foo/bar" the introspection of "/" has to be updated. A new
node has to be added at "/".
invalidate_parent_data stops invaldating the whole path because the
boolean return value of dbus_connection_get_object_path_data is used
wrong.
If we get a TRUE just go on down in the path, if FALSE is return
dbus_connection_get_object_path_data has run out of memory.
Diffstat (limited to 'gdbus/object.c')
| -rw-r--r-- | gdbus/object.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/gdbus/object.c b/gdbus/object.c index afa904e..49006ec 100644 --- a/gdbus/object.c +++ b/gdbus/object.c @@ -475,12 +475,13 @@ static void invalidate_parent_data(DBusConnection *conn, const char *child_path) if (!strlen(parent_path)) goto done; - if (!dbus_connection_get_object_path_data(conn, parent_path, - (void *) &data)) { - invalidate_parent_data(conn, parent_path); + if (dbus_connection_get_object_path_data(conn, parent_path, + (void *) &data) == FALSE) { goto done; } + invalidate_parent_data(conn, parent_path); + if (data == NULL) goto done; |