BACKPORT: sched_setattr: switch to copy_struct_from_user()
(cherry picked from commit dff3a85feceade213daf153556fd32a3b0a73c63)
Switch sched_setattr() syscall from it's own copying struct sched_attr
from userspace to the new dedicated copy_struct_from_user() helper.
The change is very straightforward, and helps unify the syscall
interface for struct-from-userspace syscalls. Ideally we could also
unify sched_getattr(2)-style syscalls as well, but unfortunately the
correct semantics for such syscalls are much less clear (see [1] for
more detail). In future we could come up with a more sane idea for how
the syscall interface should look.
[1]: commit 1251201c0d34 ("sched/core: Fix uclamp ABI bug, clean up and
robustify sched_read_attr() ABI logic and code")
Bug: 261681571
Change-Id: Ia19bddfec313fef20592569d911881e7c9aa9ddb
Signed-off-by:
Aleksa Sarai <cyphar@cyphar.com>
Reviewed-by:
Kees Cook <keescook@chromium.org>
Reviewed-by:
Christian Brauner <christian.brauner@ubuntu.com>
[christian.brauner@ubuntu.com: improve commit message]
Link: https://lore.kernel.org/r/20191001011055.19283-4-cyphar@cyphar.com
Signed-off-by:
Christian Brauner <christian.brauner@ubuntu.com>
Signed-off-by:
JohnnLee <johnnlee@google.com>
Loading
Please register or sign in to comment