Staging: bcm: Add min/max restrictions for IOCTL_BCM_REGISTER_READ_PRIVATE
authorKevin McKinney <klmckinney1@gmail.com>
Tue, 27 Sep 2011 02:03:59 +0000 (22:03 -0400)
committerGreg Kroah-Hartman <gregkh@suse.de>
Fri, 30 Sep 2011 00:34:51 +0000 (17:34 -0700)
commit0a2cc4977ffd551b58ae20c646bd7083ba5a89d2
treeda3285cfe34ffaf5047097ad18ba0334799d9b02
parentd515d0ff36a7afd528f32e3511780ad8385d957e
Staging: bcm: Add min/max restrictions for IOCTL_BCM_REGISTER_READ_PRIVATE

This patch fixes two issues within bcm/Bcmchar.c. The
first condition in the or statement checks if variable
IoBuffer.OutputLength, defined from user space, is
greater than the maximum value allowed for an
unsigned short. IoBuffer.OutputLength is then used
in a kmalloc call to return a pointer to memory. If
this size is greater than an unsigned short, it
becomes useless. The second condition in the or statement
checks if the same variable, IoBuffer.OutputLength is
equal to zero before invoking the kmalloc call. In
this case, if a zero size is sent to kmalloc, a valid
pointer to memory is returned instead of the expected NULL.

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