diff options
| author | Oleg Nesterov <oleg@redhat.com> | 2017-09-27 09:25:30 -0600 |
|---|---|---|
| committer | Kees Cook <keescook@chromium.org> | 2017-09-27 22:51:12 -0700 |
| commit | 66a733ea6b611aecf0119514d2dddab5f9d6c01e (patch) | |
| tree | 0b1d31848a156eb1008a3adb65fc716fe446153f /kernel/module_signing.c | |
| parent | e19b205be43d11bff638cad4487008c48d21c103 (diff) | |
seccomp: fix the usage of get/put_seccomp_filter() in seccomp_get_filter()
As Chris explains, get_seccomp_filter() and put_seccomp_filter() can end
up using different filters. Once we drop ->siglock it is possible for
task->seccomp.filter to have been replaced by SECCOMP_FILTER_FLAG_TSYNC.
Fixes: f8e529ed941b ("seccomp, ptrace: add support for dumping seccomp filters")
Reported-by: Chris Salls <chrissalls5@gmail.com>
Cc: stable@vger.kernel.org # needs s/refcount_/atomic_/ for v4.12 and earlier
Signed-off-by: Oleg Nesterov <oleg@redhat.com>
[tycho: add __get_seccomp_filter vs. open coding refcount_inc()]
Signed-off-by: Tycho Andersen <tycho@docker.com>
[kees: tweak commit log]
Signed-off-by: Kees Cook <keescook@chromium.org>
Diffstat (limited to 'kernel/module_signing.c')
0 files changed, 0 insertions, 0 deletions