From: Anurag Biradar Date: Tue, 6 Dec 2016 04:30:15 +0000 (+0530) Subject: bt-core: Fix bt-core crash X-Git-Tag: submit/tizen_3.0/20161207.043635 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=refs%2Ftags%2Fsubmit%2Ftizen_3.0%2F20161207.043635;hp=5a7c11ce8de623411f111768212dfb4e255fa4f0;p=platform%2Fcore%2Fconnectivity%2Fbluetooth-frwk.git bt-core: Fix bt-core crash g_dbus_proxy_call_sync function expects a double pointer for "err" variable in it's parameter list Change-Id: I534c22e1330998aa0fd9fb8e03f2d222f1c2f04a --- diff --git a/bt-core/bt-core-dbus-handler.c b/bt-core/bt-core-dbus-handler.c index c9c07c3..0d04f33 100755 --- a/bt-core/bt-core-dbus-handler.c +++ b/bt-core/bt-core-dbus-handler.c @@ -113,7 +113,7 @@ int _bt_core_start_httpproxy(void) { GVariant *variant = NULL; unsigned char enabled; - + GError *err = NULL; BT_DBG(" "); hps_gproxy = _bt_core_gdbus_get_hps_proxy(); @@ -123,7 +123,11 @@ int _bt_core_start_httpproxy(void) } variant = g_dbus_proxy_call_sync(hps_gproxy, "enable", - NULL, G_DBUS_CALL_FLAGS_NONE, -1, NULL, NULL); + NULL, G_DBUS_CALL_FLAGS_NONE, -1, NULL, &err); + if (err) { + BT_ERR("Error : %s" , err->message); + g_clear_error(&err); + } if (variant) { g_variant_get(variant, "(y)", &enabled); BT_ERR("HPS enabled status 0x%x", enabled); @@ -135,7 +139,7 @@ int _bt_core_stop_httpproxy(void) { GVariant *variant = NULL; unsigned char enabled; - + GError *err = NULL; BT_DBG(" "); hps_gproxy = _bt_core_gdbus_get_hps_proxy(); @@ -145,7 +149,11 @@ int _bt_core_stop_httpproxy(void) } variant = g_dbus_proxy_call_sync(hps_gproxy, "disable", - NULL, G_DBUS_CALL_FLAGS_NONE, -1, NULL, NULL); + NULL, G_DBUS_CALL_FLAGS_NONE, -1, NULL, &err); + if (err) { + BT_ERR("Error : %s" , err->message); + g_clear_error(&err); + } if (variant) { g_variant_get(variant, "(y)", &enabled); BT_ERR("HPS disabled status 0x%x", enabled);