copy the device name with correct buffer size

Description: This patch corrects the copying
of device/peer name using destination size, as
g_strlcpy handles the null terminations internally.

Change-Id: I89b4149bf518a02478818071d9beeccb63b324e7
Signed-off-by: Milind Murhekar <m.murhekar@samsung.com>

diff --git a/packaging/wifi-direct-manager.spec b/packaging/wifi-direct-manager.spec
index 6116930..a48d894 100644 (file)
--- a/packaging/wifi-direct-manager.spec
+++ b/packaging/wifi-direct-manager.spec
@@ -6,7 +6,7 @@
 
 Name:          wifi-direct-manager
 Summary:       Wi-Fi Direct manger
-Version:       1.2.268
+Version:       1.2.269
 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 2cee0a2..411743e 100644 (file)
--- a/plugin/wpasupplicant/ctrl_iface_dbus/wfd-plugin-wpasupplicant.c
+++ b/plugin/wpasupplicant/ctrl_iface_dbus/wfd-plugin-wpasupplicant.c
@@ -695,7 +695,7 @@ static void __ws_get_peer_property(const char *key, GVariant *value, void *user_
                const char *name = NULL;
 
                g_variant_get(value, "&s", &name);
-               g_strlcpy(peer->dev_name, name, WS_SSID_LEN);
+               g_strlcpy(peer->dev_name, name, WS_SSID_LEN + 1);
                WDP_LOGD("Device name [%s]", peer->dev_name);
 
        } else if (g_strcmp0(key, "config_method") == 0) {
@@ -801,7 +801,7 @@ static void __ws_peer_property(const char *key, GVariant *value, void *user_data
                const char *name = NULL;
 
                g_variant_get(value, "&s", &name);
-               g_strlcpy(peer->name, name, WS_SSID_LEN);
+               g_strlcpy(peer->name, name, WS_SSID_LEN + 1);
                WDP_LOGD("Device Name [%s]", peer->name);
 
        } else if (g_strcmp0(key, "config_method") == 0) {