diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2016-02-22 22:33:09 +0300 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2016-03-06 01:48:04 +0300 |
commit | e0aa762cac39d18aa60938a3d1f7f6b42fcb2cdf (patch) | |
tree | 89aaadef1ad169777f0959eb722f8a76bbc6831d /drivers/staging/gdm72xx | |
parent | deb0b53bb52a0c53c43bf894b40630b414ec56fa (diff) | |
download | linux-e0aa762cac39d18aa60938a3d1f7f6b42fcb2cdf.tar.xz |
Staging: gdm72xx: underflow bug in gdm_wimax_ioctl_get_data()
"size" here should be unsigned, otherwise we might end up trying to copy
negative bytes in gdm_wimax_ioctl_get_data() resulting in an information
leak.
Reported-by: Alan Cox <gnomes@lxorguk.ukuu.org.uk>
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/gdm72xx')
-rw-r--r-- | drivers/staging/gdm72xx/wm_ioctl.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/staging/gdm72xx/wm_ioctl.h b/drivers/staging/gdm72xx/wm_ioctl.h index 631cb1d23c7e..032cb073461c 100644 --- a/drivers/staging/gdm72xx/wm_ioctl.h +++ b/drivers/staging/gdm72xx/wm_ioctl.h @@ -74,12 +74,12 @@ struct fsm_s { }; struct data_s { - int size; + unsigned int size; void *buf; }; struct udata_s { - int size; + unsigned int size; void __user *buf; }; |