Staging: bcm: Clean up code in ioctl: IOCTL_BCM_EEPROM_REGISTER_READ
authorKevin McKinney <klmckinney1@gmail.com>
Wed, 9 Nov 2011 03:33:35 +0000 (22:33 -0500)
committerGreg Kroah-Hartman <gregkh@suse.de>
Sun, 27 Nov 2011 01:13:33 +0000 (17:13 -0800)
This patch verifies two conditions before executing
a kmalloc call. First, it checks to see that
IoBuffer.OutputLength is not greater than an
unsigned short. If so, an invalid value may be
returned. The second change is a check to make
sure IoBuffer.OutputLength is not equal to
zero. Which simply keeps this code inline with
the other ioctl, IOCTL_BCM_REGISTER_READ_PRIVATE.

Signed-off-by: Kevin McKinney <klmckinney1@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/bcm/Bcmchar.c

index e110d0e68872dfa2421d65f02440f1bab6d4f363..7cffbddd5e9cef42e3e06efc6041ad69aca48d20 100644 (file)
@@ -306,7 +306,11 @@ static long bcm_char_ioctl(struct file *filp, UINT cmd, ULONG arg)
                if (copy_from_user(&sRdmBuffer, IoBuffer.InputBuffer, IoBuffer.InputLength))
                        return -EFAULT;
 
-               /* FIXME: don't trust user supplied length */
+               if (IoBuffer.OutputLength > USHRT_MAX ||
+                       IoBuffer.OutputLength == 0) {
+                       return -EINVAL;
+               }
+
                temp_buff = kmalloc(IoBuffer.OutputLength, GFP_KERNEL);
                if (!temp_buff)
                        return STATUS_FAILURE;