Fix bugs which are detected by static analyzer

Change-Id: I4a28e45afc9930be9ce43d4764d473f832b9330f
Signed-off-by: Yu jiung <jiung.yu@samsung.com>

diff --git a/packaging/capi-network-wifi-direct.spec b/packaging/capi-network-wifi-direct.spec
index 295114b..bf0d81a 100755 (executable)
--- 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 (executable)
--- 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 (file)
--- 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;