Fix warnings from static analyzer

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

diff --git a/packaging/wifi-direct-manager.spec b/packaging/wifi-direct-manager.spec
index d847fd9..026a6cf 100755 (executable)
--- a/packaging/wifi-direct-manager.spec
+++ b/packaging/wifi-direct-manager.spec
@@ -1,6 +1,6 @@
 Name:          wifi-direct-manager
 Summary:       Wi-Fi Direct manger
-Version:       1.2.191
+Version:       1.2.192
 Release:       1
 Group:      Network & Connectivity/Wireless
 License:    Apache-2.0

diff --git a/plugin/wpasupplicant/ctrl_iface_dbus/wfd-plugin-wpasupplicant.c b/plugin/wpasupplicant/ctrl_iface_dbus/wfd-plugin-wpasupplicant.c
index 99eaabd..6aa0a57 100755 (executable)
--- a/plugin/wpasupplicant/ctrl_iface_dbus/wfd-plugin-wpasupplicant.c
+++ b/plugin/wpasupplicant/ctrl_iface_dbus/wfd-plugin-wpasupplicant.c
@@ -1642,7 +1642,8 @@ static void _ws_process_device_found_properties(GDBusConnection *connection,
        WDP_LOGD("Retrive Added path [%s]", peer_path);
 
        loc = strrchr(peer_path,'/');
-       __ws_mac_compact_to_normal(loc + 1, peer_dev);
+       if (loc != NULL)
+               __ws_mac_compact_to_normal(loc + 1, peer_dev);
        __ws_txt_to_mac(peer_dev, event.dev_addr);
        WDP_LOGD("peer mac [" MACSTR "]", MAC2STR(event.dev_addr));
 

diff --git a/src/wifi-direct-manager.c b/src/wifi-direct-manager.c
index de28b6e..3bc894a 100755 (executable)
--- a/src/wifi-direct-manager.c
+++ b/src/wifi-direct-manager.c
@@ -709,7 +709,7 @@ int wfd_manager_accept_connection(wfd_manager_s *manager, unsigned char *peer_ad
                return WIFI_DIRECT_ERROR_OPERATION_FAILED;
        }
 
-       if (!memcmp(session->peer->dev_addr, peer_addr ,MACADDR_LEN)) {
+       if (!memcmp(session->peer->dev_addr, peer_addr, MACADDR_LEN)) {
                WDS_LOGE("Peer and ongoing session peer are different");
                return WIFI_DIRECT_ERROR_OPERATION_FAILED;
        }