diff options
author | Eric Paris <eparis@redhat.com> | 2012-01-03 14:23:08 -0500 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2012-01-17 16:17:01 -0500 |
commit | 4043cde8ecf7f7d880eb1133c201a3d392fd68c3 (patch) | |
tree | d740c60e6b56565a7e996c3d0308e66f7c8651f8 /kernel | |
parent | 633b45454503489209b0d9a45f9e3cd1b852c614 (diff) |
audit: do not call audit_getname on error
Just a code cleanup really. We don't need to make a function call just for
it to return on error. This also makes the VFS function even easier to follow
and removes a conditional on a hot path.
Signed-off-by: Eric Paris <eparis@redhat.com>
Diffstat (limited to 'kernel')
-rw-r--r-- | kernel/auditsc.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/kernel/auditsc.c b/kernel/auditsc.c index bd084a13c719..9161e70a4379 100644 --- a/kernel/auditsc.c +++ b/kernel/auditsc.c | |||
@@ -1913,9 +1913,6 @@ void __audit_getname(const char *name) | |||
1913 | struct audit_context *context = current->audit_context; | 1913 | struct audit_context *context = current->audit_context; |
1914 | struct audit_names *n; | 1914 | struct audit_names *n; |
1915 | 1915 | ||
1916 | if (IS_ERR(name) || !name) | ||
1917 | return; | ||
1918 | |||
1919 | if (!context->in_syscall) { | 1916 | if (!context->in_syscall) { |
1920 | #if AUDIT_DEBUG == 2 | 1917 | #if AUDIT_DEBUG == 2 |
1921 | printk(KERN_ERR "%s:%d(:%d): ignoring getname(%p)\n", | 1918 | printk(KERN_ERR "%s:%d(:%d): ignoring getname(%p)\n", |