Re: [PATCH 20/40] mn10300: Use set_current_blocked() and block_sigmask()

From: David Howells
Date: Tue Feb 14 2012 - 10:56:37 EST


Matt Fleming <matt@xxxxxxxxxxxxxxxxx> wrote:

> asmlinkage long sys_sigsuspend(int history0, int history1, old_sigset_t mask)
> {
> - mask &= _BLOCKABLE;
> - spin_lock_irq(&current->sighand->siglock);
> + sigset_t blocked;
> +
> current->saved_sigmask = current->blocked;

Can this line create a race, now that it's not protected by a lock? It is
possible for current->blocked to get altered by another thread.

David
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/