From: Yu Date: Mon, 3 Feb 2020 04:09:02 +0000 (+0900) Subject: Fix bugs which are detected by static analyzer X-Git-Tag: submit/tizen/20200205.081631^0 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=7d7fdc711911a3587ba18457b2ca6cb29f95f8aa;p=platform%2Fcore%2Fapi%2Fwifi-direct.git Fix bugs which are detected by static analyzer Change-Id: I4a28e45afc9930be9ce43d4764d473f832b9330f Signed-off-by: Yu jiung --- diff --git a/packaging/capi-network-wifi-direct.spec b/packaging/capi-network-wifi-direct.spec index 295114b..bf0d81a 100755 --- a/packaging/capi-network-wifi-direct.spec +++ b/packaging/capi-network-wifi-direct.spec @@ -3,7 +3,7 @@ Name: capi-network-wifi-direct Summary: Network WiFi-Direct Library -Version: 1.2.101 +Version: 1.2.102 Release: 1 Group: Network & Connectivity/API License: Apache-2.0 diff --git a/src/wifi-direct-client-proxy.c b/src/wifi-direct-client-proxy.c index 44519fa..79ecc6a 100755 --- a/src/wifi-direct-client-proxy.c +++ b/src/wifi-direct-client-proxy.c @@ -3662,6 +3662,7 @@ int wifi_direct_foreach_persistent_groups(wifi_direct_persistent_group_cb cb, unsigned char mac_address[MACADDR_LEN] = {0, }; wifi_direct_dbus_unpack_ay(mac_address, var, MACADDR_LEN); + g_free(go_mac_address); go_mac_address = (char*) g_try_malloc0(MACSTR_LEN); if (go_mac_address) g_snprintf(go_mac_address, MACSTR_LEN, MACSTR, MAC2STR(mac_address)); diff --git a/test/wifi_direct_test.c b/test/wifi_direct_test.c index d4ea0a7..451b90f 100644 --- a/test/wifi_direct_test.c +++ b/test/wifi_direct_test.c @@ -2133,8 +2133,10 @@ void process_input(const char *input, gpointer user_data) break; printf("Enter info 2\n"); - if (scanf(" %50ms", &info2) < 1) + if (scanf(" %50ms", &info2) < 1) { free(info1); + info1 = NULL; + } break; case 4: @@ -2154,7 +2156,8 @@ void process_input(const char *input, gpointer user_data) result = wifi_direct_register_service(service_type, info1, info2, &input); printf("wifi_direct_register_service() service id=[%u] result=[%d]\n", input, result); - free(info1); + if (info1) + free(info1); free(info2); } break;