summaryrefslogtreecommitdiffstats
path: root/kernel/auditsc.c
diff options
context:
space:
mode:
authorDavid Woodhouse <dwmw2@shinybook.infradead.org>2005-06-24 09:21:49 +0200
committerDavid Woodhouse <dwmw2@shinybook.infradead.org>2005-06-24 09:21:49 +0200
commit993e2d4106e94dae6e8cfbeb32073bd12cdee203 (patch)
treefdb89f0a1f773795b092e0ddbb902d071f5f9c5c /kernel/auditsc.c
parentAUDIT: No really, we don't want to audit auditd. (diff)
downloadlinux-993e2d4106e94dae6e8cfbeb32073bd12cdee203.tar.xz
linux-993e2d4106e94dae6e8cfbeb32073bd12cdee203.zip
AUDIT: Return correct result from audit_filter_rules()
When the task refcounting was added to audit_filter_rules() it became more of a problem that this function was violating the 'only one return from each function' rule. In fixing it to use a variable to store 'ret' I stupidly neglected to actually change the 'return 1;' at the end. This makes it not work very well. Signed-off-by: David Woodhouse <dwmw2@infradead.org>
Diffstat (limited to 'kernel/auditsc.c')
-rw-r--r--kernel/auditsc.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/auditsc.c b/kernel/auditsc.c
index 20c7d8560af0..7b123f0a9481 100644
--- a/kernel/auditsc.c
+++ b/kernel/auditsc.c
@@ -557,7 +557,7 @@ int audit_filter_user(int pid, int type)
rcu_read_unlock();
put_task_struct(tsk);
- return 1; /* Audit by default */
+ return ret; /* Audit by default */
}