summaryrefslogtreecommitdiffstats
path: root/kernel
diff options
context:
space:
mode:
authorDavid Woodhouse <dwmw2@shinybook.infradead.org>2005-06-24 08:21:49 +0100
committerDavid Woodhouse <dwmw2@shinybook.infradead.org>2005-06-24 08:21:49 +0100
commit993e2d4106e94dae6e8cfbeb32073bd12cdee203 (patch)
treefdb89f0a1f773795b092e0ddbb902d071f5f9c5c /kernel
parent9e94e66a5bc739ab525ec0a26ba75300aaf154f3 (diff)
downloadop-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')
-rw-r--r--kernel/auditsc.c2
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 */
}
OpenPOWER on IntegriCloud