The patch titled Subject: signal: make flush_sigqueue_mask() void has been added to the -mm tree. Its filename is signal-make-flush_sigqueue_mask-void.patch This patch should soon appear at http://ozlabs.org/~akpm/mmots/broken-out/signal-make-flush_sigqueue_mask-void.patch and later at http://ozlabs.org/~akpm/mmotm/broken-out/signal-make-flush_sigqueue_mask-void.patch Before you just go and hit "reply", please: a) Consider who else should be cc'ed b) Prefer to cc a suitable mailing list as well c) Ideally: find the original patch on the mailing list and do a reply-to-all to that, adding suitable additional cc's *** Remember to use Documentation/process/submit-checklist.rst when testing your code *** The -mm tree is included into linux-next and is updated there every 3-4 working days ------------------------------------------------------ From: Christian Brauner <christian@xxxxxxxxxx> Subject: signal: make flush_sigqueue_mask() void The return value of flush_sigqueue_mask() is never checked anywhere. Link: http://lkml.kernel.org/r/20180602103653.18181-14-christian@xxxxxxxxxx Signed-off-by: Christian Brauner <christian@xxxxxxxxxx> Reviewed-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> Cc: Al Viro <viro@xxxxxxxxxxxxxxxxxx> Cc: Eric W. Biederman <ebiederm@xxxxxxxxxxxx> Cc: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> Cc: Ingo Molnar <mingo@xxxxxxxxxx> Cc: James Morris <james.morris@xxxxxxxxxxxxx> Cc: Kees Cook <keescook@xxxxxxxxxxxx> Cc: Oleg Nesterov <oleg@xxxxxxxxxx> Cc: Peter Zijlstra <peterz@xxxxxxxxxxxxx> Cc: Stephen Smalley <sds@xxxxxxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- kernel/signal.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff -puN kernel/signal.c~signal-make-flush_sigqueue_mask-void kernel/signal.c --- a/kernel/signal.c~signal-make-flush_sigqueue_mask-void +++ a/kernel/signal.c @@ -687,14 +687,14 @@ void signal_wake_up_state(struct task_st * * All callers must be holding the siglock. */ -static int flush_sigqueue_mask(sigset_t *mask, struct sigpending *s) +static void flush_sigqueue_mask(sigset_t *mask, struct sigpending *s) { struct sigqueue *q, *n; sigset_t m; sigandsets(&m, mask, &s->signal); if (sigisemptyset(&m)) - return 0; + return; sigandnsets(&s->signal, &s->signal, mask); list_for_each_entry_safe(q, n, &s->list, list) { @@ -703,7 +703,6 @@ static int flush_sigqueue_mask(sigset_t __sigqueue_free(q); } } - return 1; } static inline int is_si_special(const struct siginfo *info) _ Patches currently in -mm which might be from christian@xxxxxxxxxx are signal-make-force_sigsegv-void.patch signal-make-kill_as_cred_perm-return-bool.patch signal-make-may_ptrace_stop-return-bool.patch signal-make-do_sigpending-void.patch signal-simplify-rt_sigaction.patch signal-make-kill_ok_by_cred-return-bool.patch signal-make-sig_handler_ignored-return-bool.patch signal-make-sig_task_ignored-return-bool.patch signal-make-sig_ignored-return-bool.patch signal-make-has_pending_signals-return-bool.patch signal-make-recalc_sigpending_tsk-return-bool.patch signal-make-unhandled_signal-return-bool.patch signal-make-flush_sigqueue_mask-void.patch signal-make-wants_signal-return-bool.patch signal-make-legacy_queue-return-bool.patch signal-make-sigkill_pending-return-bool.patch signal-make-get_signal-return-bool.patch -- To unsubscribe from this list: send the line "unsubscribe mm-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html