staging: lustre: use memdup_user to allocate memory and copy from user
Replace a call to kmalloc and a call to copy_from_user with a call to memdup_user to simplify the code. Issue found with coccicheck. Signed-off-by: Dafna Hirschfeld <dafna3@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
f1345b2f21
commit
2baddf262e
@ -648,12 +648,10 @@ static int lst_test_add_ioctl(struct lstio_test_args *args)
|
||||
return -EINVAL;
|
||||
|
||||
if (args->lstio_tes_param) {
|
||||
param = kmalloc(args->lstio_tes_param_len, GFP_KERNEL);
|
||||
if (!param)
|
||||
goto out;
|
||||
if (copy_from_user(param, args->lstio_tes_param,
|
||||
args->lstio_tes_param_len)) {
|
||||
rc = -EFAULT;
|
||||
param = memdup_user(args->lstio_tes_param,
|
||||
args->lstio_tes_param_len);
|
||||
if (IS_ERR(param)) {
|
||||
rc = PTR_ERR(param);
|
||||
goto out;
|
||||
}
|
||||
}
|
||||
|
Loading…
x
Reference in New Issue
Block a user