From 64b82b1934ffc3eea4f25acda8e7e797879abd14 Mon Sep 17 00:00:00 2001 From: Anurag Biradar Date: Tue, 6 Dec 2016 10:00:15 +0530 Subject: [PATCH] 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 --- bt-core/bt-core-dbus-handler.c | 16 ++++++++++++---- 1 file changed, 12 insertions(+), 4 deletions(-) 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); -- 2.7.4