summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPatrik Flykt <patrik.flykt@linux.intel.com>2013-01-28 17:44:43 +0200
committerPatrik Flykt <patrik.flykt@linux.intel.com>2013-01-30 10:28:46 +0200
commite49cb1f4dedfba7e6dc465ebdf46716b497587fb (patch)
tree287d844d3f195c18d0739eef04f549ca4e36a177
parentba18b4a289ff18c094d544762edfdc303c4ffc90 (diff)
downloadconnman-e49cb1f4dedfba7e6dc465ebdf46716b497587fb.tar.gz
connman-e49cb1f4dedfba7e6dc465ebdf46716b497587fb.tar.bz2
connman-e49cb1f4dedfba7e6dc465ebdf46716b497587fb.zip
service: Must not return NULL with a synchronous GDBus method call
If NULL is returned, GDBus will call the registered function repeatedly.
-rw-r--r--src/service.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/src/service.c b/src/service.c
index 1e7cec78..a1d4a2f5 100644
--- a/src/service.c
+++ b/src/service.c
@@ -3765,8 +3765,6 @@ static DBusMessage *disconnect_service(DBusConnection *conn,
if (err < 0) {
if (err != -EINPROGRESS)
return __connman_error_failed(msg, -err);
-
- return NULL;
}
return g_dbus_create_reply(msg, DBUS_TYPE_INVALID);