diff options
author | David Woodhouse <dwmw2@shinybook.infradead.org> | 2005-06-24 03:21:49 -0400 |
---|---|---|
committer | David Woodhouse <dwmw2@shinybook.infradead.org> | 2005-06-24 03:21:49 -0400 |
commit | 993e2d4106e94dae6e8cfbeb32073bd12cdee203 (patch) | |
tree | fdb89f0a1f773795b092e0ddbb902d071f5f9c5c /kernel/auditsc.c | |
parent | 9e94e66a5bc739ab525ec0a26ba75300aaf154f3 (diff) |
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) | |||
557 | rcu_read_unlock(); | 557 | rcu_read_unlock(); |
558 | put_task_struct(tsk); | 558 | put_task_struct(tsk); |
559 | 559 | ||
560 | return 1; /* Audit by default */ | 560 | return ret; /* Audit by default */ |
561 | 561 | ||
562 | } | 562 | } |
563 | 563 | ||