Commit 11f80ddf authored by Güngör Erseymen's avatar Güngör Erseymen Committed by Greg Kroah-Hartman

staging: comedi: remove 'bc.subdevice < 0' check in do_bufconfig_ioctl()

Type of variable 'bc' is struct comedi_bufconfig and 'subdevice' is
defined as unsigned, so there is no need to check if it is less then
zero.
Signed-off-by: default avatarGüngör Erseymen <gelurine@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent fa45f2c7
...@@ -660,7 +660,7 @@ static int do_bufconfig_ioctl(struct comedi_device *dev, ...@@ -660,7 +660,7 @@ static int do_bufconfig_ioctl(struct comedi_device *dev,
if (copy_from_user(&bc, arg, sizeof(bc))) if (copy_from_user(&bc, arg, sizeof(bc)))
return -EFAULT; return -EFAULT;
if (bc.subdevice >= dev->n_subdevices || bc.subdevice < 0) if (bc.subdevice >= dev->n_subdevices)
return -EINVAL; return -EINVAL;
s = &dev->subdevices[bc.subdevice]; s = &dev->subdevices[bc.subdevice];
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment