Yama: access task_struct->comm directly
authorKees Cook <keescook@chromium.org>
Wed, 15 Aug 2012 18:41:55 +0000 (11:41 -0700)
committerJames Morris <james.l.morris@oracle.com>
Fri, 17 Aug 2012 10:40:38 +0000 (20:40 +1000)
commit0e484cc0f33562a5261f1bab5a2b24ee0049c5f9
treed73acd2bc97866b256a70acb83e4ca69bce6e32d
parent2f2a0c5f4b60dcf2d8db567191c7178e461721c8
Yama: access task_struct->comm directly

The core ptrace access checking routine holds a task lock, and when
reporting a failure, Yama takes a separate task lock. To avoid a
potential deadlock with two ptracers taking the opposite locks, do not
use get_task_comm() and just use ->comm directly since accuracy is not
important for the report.

Reported-by: Fengguang Wu <fengguang.wu@intel.com>
Suggested-by: Oleg Nesterov <oleg@redhat.com>
CC: stable@vger.kernel.org
Signed-off-by: Kees Cook <keescook@chromium.org>
Acked-by: John Johansen <john.johansen@canonical.com>
Signed-off-by: James Morris <james.l.morris@oracle.com>
security/yama/yama_lsm.c