forked from luck/tmp_suning_uos_patched
esas2r: don't bother with __copy_to_user()
sure, we'd done copy_from_user() on the same range, so we can skip access_ok()... and it's not worth bothering. Just use copy_to_user(). Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
This commit is contained in:
parent
8f3d9f3542
commit
3d3185ae59
|
@ -1510,7 +1510,7 @@ int esas2r_ioctl_handler(void *hostdata, unsigned int cmd, void __user *arg)
|
|||
}
|
||||
|
||||
/* Always copy the buffer back, if only to pick up the status */
|
||||
err = __copy_to_user(arg, ioctl, sizeof(struct atto_express_ioctl));
|
||||
err = copy_to_user(arg, ioctl, sizeof(struct atto_express_ioctl));
|
||||
if (err != 0) {
|
||||
esas2r_log(ESAS2R_LOG_WARN,
|
||||
"ioctl_handler copy_to_user didn't copy everything (err %d, cmd %u)",
|
||||
|
|
Loading…
Reference in New Issue
Block a user