diff options
| author | 2015-01-06 12:57:08 +0300 | |
|---|---|---|
| committer | 2015-01-17 15:58:30 -0800 | |
| commit | ef4356bf0cf7ed2c9aed45dd64d73538da023718 (patch) | |
| tree | 219d275385ec5b9fde9651cd799f57f1eea95b13 /scripts/patch-kernel | |
| parent | staging: lustre: libcfs: fix sparse warnings about static declaration (diff) | |
staging: lustre: potential underflow in mdc_iocontrol()
Smatch complains that "data->ioc_plen2" is a user controlled value and,
since we cast to signed int, the limit check can underflow. It's not
very serious because probably the copy_to_user() would return -EFAULT
on every arch that matters instead of creating an info leak. Also I
haven't followed it through to see if the value is really user
controlled.
But definitely it would be safer to cast to unsigned so let's do that.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'scripts/patch-kernel')
0 files changed, 0 insertions, 0 deletions
