• Matt Fleming's avatar
    ARM: use set_current_blocked() and block_sigmask() · 101d9b0d
    Matt Fleming authored
    As described in e6fa16ab ("signal: sigprocmask() should do
    retarget_shared_pending()") the modification of current->blocked is
    incorrect as we need to check for shared signals we're about to block.
    Also, use the new helper function introduced in commit 5e6292c0
    ("signal: add block_sigmask() for adding sigmask to current->blocked")
    which centralises the code for updating current->blocked after
    successfully delivering a signal and reduces the amount of duplicate code
    across architectures.  In the past some architectures got this code wrong,
    so using this helper function should stop that from happening again.
    Cc: Arnd Bergmann <arnd.bergmann@linaro.org>
    Cc: Dave Martin <dave.martin@linaro.org>
    Cc: Nicolas Pitre <nicolas.pitre@linaro.org>
    Cc: Will Deacon <will.deacon@arm.com>
    Acked-by: default avatarOleg Nesterov <oleg@redhat.com>
    Signed-off-by: default avatarMatt Fleming <matt.fleming@intel.com>
    Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
    Signed-off-by: default avatarRussell King <rmk+kernel@arm.linux.org.uk>
signal.c 21.3 KB