AUDIT: Return correct result from audit_filter_rules()
authorDavid Woodhouse <dwmw2@shinybook.infradead.org>
Fri, 24 Jun 2005 07:21:49 +0000 (08:21 +0100)
committerDavid Woodhouse <dwmw2@shinybook.infradead.org>
Fri, 24 Jun 2005 07:21:49 +0000 (08:21 +0100)
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>
kernel/auditsc.c

index 20c7d85..7b123f0 100644 (file)
@@ -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 */
 
 }