diff mbox

[DBus] Restrict DeviceName size to 32 Characters in setter

Message ID 1025785155.1132301452855488024.JavaMail.weblogic@ep2mlwas08a
State Accepted
Headers show

Commit Message

Purushottam Kushwaha Jan. 15, 2016, 10:58 a.m. UTC 
Signed-off-by: Purushottam Kushwaha <p.kushwaha@samsung.com>
---
 wpa_supplicant/dbus/dbus_new_handlers_p2p.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Jouni Malinen Jan. 15, 2016, 6:52 p.m. UTC | #1 
Thanks, applied.
diff mbox

Patch

diff --git a/wpa_supplicant/dbus/dbus_new_handlers_p2p.c b/wpa_supplicant/dbus/dbus_new_handlers_p2p.c
index e2f0afc..80cea63 100644
--- a/wpa_supplicant/dbus/dbus_new_handlers_p2p.c
+++ b/wpa_supplicant/dbus/dbus_new_handlers_p2p.c
@@ -945,7 +945,8 @@  dbus_bool_t wpas_dbus_setter_p2p_device_config(
 		if (os_strcmp(entry.key, "DeviceName") == 0) {
 			char *devname;
 
-			if (entry.type != DBUS_TYPE_STRING)
+			if (entry.type != DBUS_TYPE_STRING ||
+			    os_strlen(entry.str_value) > 32)
 				goto error;
 
 			devname = os_strdup(entry.str_value);