diff options
author | Al Viro <viro@zeniv.linux.org.uk> | 2017-12-28 14:43:06 -0500 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2017-12-28 14:43:06 -0500 |
commit | fc1c428eb46af8183be771d2c78b3902acbeffe3 (patch) | |
tree | 01ef00f78f0f7cb39e8ed55b43f50eacfba3ed2e /drivers/ata/libata-acpi.c | |
parent | 3d26759c09da7242db6ae18ce9a0f6b09bfeafcc (diff) | |
download | linux-fc1c428eb46af8183be771d2c78b3902acbeffe3.tar.bz2 |
usx2y: don't bother with access_ok() in ->dsp_load()
memdup_user() checks it, so the only effect would be failing with
-EINVAL instead of -EFAULT in case when access_ok() is false.
However, the caller has already checked access_ok() itself (and
would have buggered off with -EFAULT), so the check is completely
pointless. Removing it both simplifies the only instance
of ->dsp_load() and allows to get rid of the check in caller -
its sole effect used to be in preventing a bogus error value
from access_ok() in the instance. Let memdup_user() do the
right thing instead...
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'drivers/ata/libata-acpi.c')
0 files changed, 0 insertions, 0 deletions