diff options
| author | Al Viro <viro@zeniv.linux.org.uk> | 2020-05-07 23:28:15 -0400 |
|---|---|---|
| committer | Al Viro <viro@zeniv.linux.org.uk> | 2020-05-29 11:06:38 -0400 |
| commit | 87c233b8158a20a9c9ab1da96cb5cb1734d9006e (patch) | |
| tree | 21d71eac20c87fe59b4fd914b64a580c7fb44074 | |
| parent | fd8ec4dd4a79d5ad51d31d4a7664b5f82d108316 (diff) | |
| download | linux-87c233b8158a20a9c9ab1da96cb5cb1734d9006e.tar.bz2 | |
vmci_host: get rid of pointless access_ok()
get_user_pages_fast() doesn't need the caller to check that.
NB: reachable only from ioctl(2) and only under USER_DS
Reviewed-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
| -rw-r--r-- | drivers/misc/vmw_vmci/vmci_host.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/misc/vmw_vmci/vmci_host.c b/drivers/misc/vmw_vmci/vmci_host.c index ce16d6b99295..2d8328d928d5 100644 --- a/drivers/misc/vmw_vmci/vmci_host.c +++ b/drivers/misc/vmw_vmci/vmci_host.c @@ -233,8 +233,6 @@ static int vmci_host_setup_notify(struct vmci_ctx *context, * about the size. */ BUILD_BUG_ON(sizeof(bool) != sizeof(u8)); - if (!access_ok((void __user *)uva, sizeof(u8))) - return VMCI_ERROR_GENERIC; /* * Lock physical page backing a given user VA. |