diff options
author | Al Viro <viro@zeniv.linux.org.uk> | 2020-02-15 20:09:14 +0300 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2020-03-19 03:04:00 +0300 |
commit | c63aad695dceb08290be7845052d7d3f1d457416 (patch) | |
tree | 5833d226e6578bf4cc8ac702c632ce1ae3dfaf14 /arch/x86/ia32 | |
parent | 4b842e4e25b12951fa10dedb4bc16bc47e3b850c (diff) | |
download | linux-c63aad695dceb08290be7845052d7d3f1d457416.tar.xz |
vm86: get rid of get_user_ex() use
Just do a copyin of what we want into a local variable and
be done with that. We are guaranteed to be on shallow stack
here...
Note that conditional expression for range passed to access_ok()
in mainline had been pointless all along - the only difference
between vm86plus_struct and vm86_struct is that the former has
one extra field in the end and when we get to copyin of that
field (conditional upon 'plus' argument), we use copy_from_user().
Moreover, all fields starting with ->int_revectored are copied
that way, so we only need that check (be it done by access_ok()
or by user_access_begin()) only on the beginning of the structure -
the fields that used to be covered by that get_user_try() block.
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'arch/x86/ia32')
0 files changed, 0 insertions, 0 deletions