diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2014-12-22 10:52:39 +0300 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2015-01-17 15:46:22 -0800 |
commit | 3b66ea435234aab8fd8f389e929baedb556726c4 (patch) | |
tree | acdaffe2b8d8f46d6ba11f62d25102de2cbe479d /drivers/staging/lustre/lnet | |
parent | 466fc15cb03dfbf77f871dacc3d0255ceeee8962 (diff) |
Staging: lustre: error handling tweaks in obd_ioctl_getdata()
1) The places which called copy_from_user() were returning the number
of bytes not copied instead of -EFAULT.
2) The user could trigger a memory leak if the condition
"(hdr.ioc_len != data->ioc_len)" was true. Instead of adding a new call
to OBD_FREE_LARGE(), I created a free_buf label and changed everything
to use that label.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/lustre/lnet')
0 files changed, 0 insertions, 0 deletions