]> git.karo-electronics.de Git - mv-sheeva.git/commit
[SCSI] gdth: integer overflow in ioctl
authorDan Carpenter <error27@gmail.com>
Fri, 8 Oct 2010 07:03:07 +0000 (09:03 +0200)
committerJames Bottomley <James.Bottomley@suse.de>
Mon, 25 Oct 2010 20:01:14 +0000 (15:01 -0500)
commitf63ae56e4e97fb12053590e41a4fa59e7daa74a4
treee5087c68ff1c9387a8bed84e4b2ab30d19e02852
parent546ae796bfac6399e30da4b5af2cf7a6d0f8a4ec
[SCSI] gdth: integer overflow in ioctl

gdth_ioctl_alloc() takes the size variable as an int.
copy_from_user() takes the size variable as an unsigned long.
gen.data_len and gen.sense_len are unsigned longs.
On x86_64 longs are 64 bit and ints are 32 bit.

We could pass in a very large number and the allocation would truncate
the size to 32 bits and allocate a small buffer.  Then when we do the
copy_from_user(), it would result in a memory corruption.

CC: stable@kernel.org
Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
drivers/scsi/gdth.c