diff options
| author | David Woodhouse <dwmw2@shinybook.infradead.org> | 2005-06-24 08:21:49 +0100 |
|---|---|---|
| committer | David Woodhouse <dwmw2@shinybook.infradead.org> | 2005-06-24 08:21:49 +0100 |
| commit | 993e2d4106e94dae6e8cfbeb32073bd12cdee203 (patch) | |
| tree | fdb89f0a1f773795b092e0ddbb902d071f5f9c5c /kernel/auditsc.c | |
| parent | 9e94e66a5bc739ab525ec0a26ba75300aaf154f3 (diff) | |
| download | op-kernel-dev-993e2d4106e94dae6e8cfbeb32073bd12cdee203.zip op-kernel-dev-993e2d4106e94dae6e8cfbeb32073bd12cdee203.tar.gz | |
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 20c7d85..7b123f0 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 */ } |
