wireless: add input validation for processes
authorFelix Fietkau <nbd@openwrt.org>
Sun, 1 Dec 2013 09:53:13 +0000 (10:53 +0100)
committerFelix Fietkau <nbd@openwrt.org>
Sun, 1 Dec 2013 09:53:13 +0000 (10:53 +0100)
Signed-off-by: Felix Fietkau <nbd@openwrt.org>
wireless.c

index 7cc832dfbff0909fe1676a8090c2ac9ea506a332..273d54bd93d510664ddfb2943c730b59a4368328 100644 (file)
@@ -709,6 +709,7 @@ wireless_device_add_process(struct wireless_device *wdev, struct blob_attr *data
        struct blob_attr *tb[__PROC_ATTR_MAX];
        struct wireless_process *proc;
        char *name;
+       int pid;
 
        if (!data)
                return UBUS_STATUS_INVALID_ARGUMENT;
@@ -717,10 +718,14 @@ wireless_device_add_process(struct wireless_device *wdev, struct blob_attr *data
        if (!tb[PROC_ATTR_PID] || !tb[PROC_ATTR_EXE])
                return UBUS_STATUS_INVALID_ARGUMENT;
 
+       pid = blobmsg_get_u32(tb[PROC_ATTR_PID]);
+       if (pid < 2)
+               return UBUS_STATUS_INVALID_ARGUMENT;
+
        proc = calloc_a(sizeof(*proc),
                &name, strlen(blobmsg_data(tb[PROC_ATTR_EXE])) + 1);
 
-       proc->pid = blobmsg_get_u32(tb[PROC_ATTR_PID]);
+       proc->pid = pid;
        proc->exe = strcpy(name, blobmsg_data(tb[PROC_ATTR_EXE]));
 
        if (tb[PROC_ATTR_REQUIRED])