Fixes a problem that uses gdbus incorrectly 34/201534/1 accepted/tizen/unified/20190320.082234 submit/tizen/20190319.020059
authorINSUN PYO <insun.pyo@samsung.com>
Fri, 15 Mar 2019 08:43:58 +0000 (17:43 +0900)
committerINSUN PYO <insun.pyo@samsung.com>
Fri, 15 Mar 2019 08:48:09 +0000 (17:48 +0900)
When you repeatedly plug and unplug microUSB on TM1, you can find below journal log intermittently.

/////////////////// journal message //////////////////////////////////////
Jan 01 15:16:17 localhost dbus[329]: [system] Rejected send message, 0 matched rules; type="error", sender=":1.26" (uid=451 pid=511 comm="") interface="(unset)" member="(unset)" error name="org.freedesktop.DBus.Error.UnknownMethod" requested_reply="0" destination=":1.32" privilege="(n/a)" (uid=451 pid=546 comm="")
Jan 01 15:16:17 localhost dbus[329]: [system] Rejected send message, 0 matched rules; type="error", sender=":1.26" (uid=451 pid=511 comm="") interface="(unset)" member="(unset)" error name="org.freedesktop.DBus.Error.UnknownMethod" requested_reply="0" destination=":1.32" privilege="(n/a)" (uid=451 pid=546 comm="")
Jan 01 15:16:17 localhost dbus[329]: [system] Rejected send message, 0 matched rules; type="error", sender=":1.26" (uid=451 pid=511 comm="") interface="(unset)" member="(unset)" error name="org.freedesktop.DBus.Error.UnknownMethod" requested_reply="0" destination=":1.32" privilege="(n/a)" (uid=451 pid=546 comm="")
//////////////////////////////////////////////////////////////////////////

Change-Id: I56b440a7c2b8a680f83234be9d7c6e8859ef4d4f

src/daemon/mm_resource_manager_daemon_dbus.c

index aeaf80d258ac291fa6480a63732e1b91c0675ff3..9169a240f7e657d12f826166ad4db2c54464f9b5 100755 (executable)
@@ -194,7 +194,7 @@ static gboolean on_get_conf(MMResourceManager *interface,
        mmresource_manager_complete_conf(interface, invocation, error,
                        res_type_conf, res_cond_conf);
 
-       return FALSE;
+       return TRUE;
 }
 
 static gboolean on_create_handle(MMResourceManager *interface,
@@ -210,7 +210,7 @@ static gboolean on_create_handle(MMResourceManager *interface,
 
        mmresource_manager_complete_create(interface, invocation, id, error);
 
-       return FALSE;
+       return TRUE;
 }
 
 static gboolean on_destroy_handle(MMResourceManager *interface,
@@ -222,7 +222,7 @@ static gboolean on_destroy_handle(MMResourceManager *interface,
        error = _mmrm_dmn_dbus_destroy(id);
 
        mmresource_manager_complete_destroy(interface, invocation, error);
-       return FALSE;
+       return TRUE;
 }
 
 static gboolean on_commit_handle(MMResourceManager *interface,
@@ -258,7 +258,7 @@ static gboolean on_commit_handle(MMResourceManager *interface,
        g_variant_iter_free(release_array);
        g_variant_iter_free(acquire_array);
 
-       return FALSE;
+       return TRUE;
 }
 
 static void on_bus_acquired(GDBusConnection *connection, const gchar *name,
@@ -267,8 +267,7 @@ static void on_bus_acquired(GDBusConnection *connection, const gchar *name,
        GError* error = NULL;
        MM_RM_DEBUG("name of bus = %s, userID = %d", name, getuid());
 
-       g_signal_connect(interface, "handle-conf",
-                       G_CALLBACK(on_get_conf), NULL);
+       g_signal_connect(interface, "handle-conf", G_CALLBACK(on_get_conf), NULL);
        g_signal_connect(interface, "handle-create", G_CALLBACK(on_create_handle), NULL);
        g_signal_connect(interface, "handle-destroy", G_CALLBACK(on_destroy_handle), NULL);
        g_signal_connect(interface, "handle-commit", G_CALLBACK(on_commit_handle), NULL);