AppArmor: fix oops in apparmor_setprocattr
authorKees Cook <kees.cook@canonical.com>
Tue, 31 May 2011 18:31:41 +0000 (11:31 -0700)
committerSteve Conklin <sconklin@canonical.com>
Fri, 15 Jul 2011 17:21:13 +0000 (12:21 -0500)
commitebc769b7b56dbca2541655eb3429b12327afc7b9
treed2932a78d938a2a12af507a44b2af1083bfb6b40
parent9993ee015d68dcc2cd2ab0e612faf49974a83b0e
AppArmor: fix oops in apparmor_setprocattr

BugLink: http://bugs.launchpad.net/bugs/793702

commit a5b2c5b2ad5853591a6cac6134cd0f599a720865 upstream.

When invalid parameters are passed to apparmor_setprocattr a NULL deref
oops occurs when it tries to record an audit message. This is because
it is passing NULL for the profile parameter for aa_audit. But aa_audit
now requires that the profile passed is not NULL.

Fix this by passing the current profile on the task that is trying to
setprocattr.

Signed-off-by: Kees Cook <kees@ubuntu.com>
Signed-off-by: John Johansen <john.johansen@canonical.com>
Signed-off-by: James Morris <jmorris@namei.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Signed-off-by: Tim Gardner <tim.gardner@canonical.com>
security/apparmor/lsm.c