summaryrefslogtreecommitdiffstats
path: root/sound/ac97
diff options
context:
space:
mode:
authorAl Viro <viro@zeniv.linux.org.uk>2017-12-28 20:43:06 +0100
committerAl Viro <viro@zeniv.linux.org.uk>2017-12-28 20:43:06 +0100
commitfc1c428eb46af8183be771d2c78b3902acbeffe3 (patch)
tree01ef00f78f0f7cb39e8ed55b43f50eacfba3ed2e /sound/ac97
parentr128: don't open-code memdup_user() (diff)
downloadlinux-fc1c428eb46af8183be771d2c78b3902acbeffe3.tar.xz
linux-fc1c428eb46af8183be771d2c78b3902acbeffe3.zip
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 'sound/ac97')
0 files changed, 0 insertions, 0 deletions