Staging: batman-adv: Dont deactivate aggregation on wrong input
authorLinus Lüssing <linus.luessing@web.de>
Sat, 2 Jan 2010 10:30:52 +0000 (11:30 +0100)
committerGreg Kroah-Hartman <gregkh@suse.de>
Thu, 4 Mar 2010 00:42:36 +0000 (16:42 -0800)
A non-integer changes the aggregation mode. Therefore this patch changes
the behaviour to explicitly check strict_strtoul()'s return code.

Signed-off-by: Linus Lüssing <linus.luessing@web.de>
Acked-by: Simon Wunderlich <siwu@hrz.tu-chemnitz.de>
Signed-off-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/batman-adv/proc.c

index 4d199351c2b4a404e36c2e8c729767c45dfe1c7b..68c36b097a07c0cc5c2cc9c584ad6e8a18de29b1 100644 (file)
@@ -427,6 +427,7 @@ static ssize_t proc_aggr_write(struct file *file, const char __user *buffer,
        char *aggr_string;
        int not_copied = 0;
        unsigned long aggregation_enabled_tmp;
+       int retval;
 
        aggr_string = kmalloc(count, GFP_KERNEL);
 
@@ -436,22 +437,21 @@ static ssize_t proc_aggr_write(struct file *file, const char __user *buffer,
        not_copied = copy_from_user(aggr_string, buffer, count);
        aggr_string[count - not_copied - 1] = 0;
 
-       strict_strtoul(aggr_string, 10, &aggregation_enabled_tmp);
+       retval = strict_strtoul(aggr_string, 10, &aggregation_enabled_tmp);
 
-       if ((aggregation_enabled_tmp != 0) && (aggregation_enabled_tmp != 1)) {
+       if (retval || aggregation_enabled_tmp > 1) {
                printk(KERN_ERR "batman-adv:Aggregation can only be enabled (1) or disabled (0), given value: %li\n", aggregation_enabled_tmp);
-               goto end;
+       } else {
+               printk(KERN_INFO "batman-adv:Changing aggregation from: %s (%i) to: %s (%li)\n",
+                      (atomic_read(&aggregation_enabled) == 1 ?
+                       "enabled" : "disabled"),
+                      atomic_read(&aggregation_enabled),
+                      (aggregation_enabled_tmp == 1 ? "enabled" : "disabled"),
+                      aggregation_enabled_tmp);
+               atomic_set(&aggregation_enabled,
+                          (unsigned)aggregation_enabled_tmp);
        }
 
-       printk(KERN_INFO "batman-adv:Changing aggregation from: %s (%i) to: %s (%li)\n",
-              (atomic_read(&aggregation_enabled) == 1 ?
-               "enabled" : "disabled"),
-              atomic_read(&aggregation_enabled),
-              (aggregation_enabled_tmp == 1 ? "enabled" : "disabled"),
-              aggregation_enabled_tmp);
-
-       atomic_set(&aggregation_enabled, (unsigned)aggregation_enabled_tmp);
-end:
        kfree(aggr_string);
        return count;
 }