From: Seung-Woo Kim Date: Fri, 23 Sep 2022 02:09:52 +0000 (+0900) Subject: pass: thermal: Fix to return proper value for dbus signal handler X-Git-Tag: accepted/tizen/7.0/unified/20221110.063429^0 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=3332766d0c347e5e1a56f0370a87a378f8216d1d;p=platform%2Fcore%2Fsystem%2Fpass.git pass: thermal: Fix to return proper value for dbus signal handler From the dbus-stub generated code, signal handler should return TRUE if it handles invocation. Even ret value is error, fix to return TRUE after handling invocation. Below is dbus-stub comment about signal handler: If a signal handler returns %TRUE, it means the signal handler will handle the invocation and no other signal handlers will run. If no signal handler handles the invocation, the %G_DBUS_ERROR_UNKNOWN_METHOD error is returned. Returns: %G_DBUS_METHOD_INVOCATION_HANDLED or %TRUE if the invocation was handled, %G_DBUS_METHOD_INVOCATION_UNHANDLED or %FALSE to let other signal handlers run. NOTE: GLIB related asan heap-buffer-overflow issue is from the wrong return. Change-Id: I4db508aafba556406dce63bb0309d55c9f8a4696 Suggested-by: Sangjung Woo Signed-off-by: Seung-Woo Kim --- diff --git a/src/thermal/thermal.c b/src/thermal/thermal.c index 60440ab..8d8cb85 100644 --- a/src/thermal/thermal.c +++ b/src/thermal/thermal.c @@ -216,7 +216,7 @@ static gboolean dbus_cb_thermal_start(SystemThermal *obj, _E("failed to initialize Thermal Monitor of the daemon " "in dbus callback for a start message\n"); - return FALSE; + return TRUE; } /**