diff options
author | Christophe Leroy <christophe.leroy@c-s.fr> | 2020-04-03 09:20:51 +0200 |
---|---|---|
committer | Michael Ellerman <mpe@ellerman.id.au> | 2020-05-01 04:35:21 +0200 |
commit | 41cd780524674082b037e7c8461f90c5e42103f0 (patch) | |
tree | eb705efdf71be777efedd2cd76e1f47d7cd753e2 /lib/strncpy_from_user.c | |
parent | uaccess: Add user_read_access_begin/end and user_write_access_begin/end (diff) | |
download | linux-41cd780524674082b037e7c8461f90c5e42103f0.tar.xz linux-41cd780524674082b037e7c8461f90c5e42103f0.zip |
uaccess: Selectively open read or write user access
When opening user access to only perform reads, only open read access.
When opening user access to only perform writes, only open write
access.
Signed-off-by: Christophe Leroy <christophe.leroy@c-s.fr>
Reviewed-by: Kees Cook <keescook@chromium.org>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Link: https://lore.kernel.org/r/2e73bc57125c2c6ab12a587586a4eed3a47105fc.1585898438.git.christophe.leroy@c-s.fr
Diffstat (limited to 'lib/strncpy_from_user.c')
-rw-r--r-- | lib/strncpy_from_user.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/strncpy_from_user.c b/lib/strncpy_from_user.c index 706020b06617..b90ec550183a 100644 --- a/lib/strncpy_from_user.c +++ b/lib/strncpy_from_user.c @@ -116,9 +116,9 @@ long strncpy_from_user(char *dst, const char __user *src, long count) kasan_check_write(dst, count); check_object_size(dst, count, false); - if (user_access_begin(src, max)) { + if (user_read_access_begin(src, max)) { retval = do_strncpy_from_user(dst, src, count, max); - user_access_end(); + user_read_access_end(); return retval; } } |