diff options
author | David Woodhouse <dwmw2@shinybook.infradead.org> | 2005-06-24 09:21:49 +0200 |
---|---|---|
committer | David Woodhouse <dwmw2@shinybook.infradead.org> | 2005-06-24 09:21:49 +0200 |
commit | 993e2d4106e94dae6e8cfbeb32073bd12cdee203 (patch) | |
tree | fdb89f0a1f773795b092e0ddbb902d071f5f9c5c /kernel/auditsc.c | |
parent | AUDIT: No really, we don't want to audit auditd. (diff) | |
download | linux-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.c | 2 |
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 */ } |