audit 0.9.12 released

David Woodhouse dwmw2 at infradead.org
Fri Jun 24 07:19:23 UTC 2005


On Thu, 2005-06-23 at 17:25 -0400, Steve Grubb wrote:
> > I'm more interested in finding the real reason why it didn't work. Were
> > you setting the syscall bitmask to all ones in auditctl?
> 
> Yes.

You pointed out the problem on IRC -- well spotted. Once I'd added the
task refcounting to audit_filter_user() it became more of a problem that
the function was violating the "only one return from each function"
rule. I fixed it to use a variable for the return code and always reach
the end, but in a fit of stupidity I left the 'return 1' at the end :)

audit.68 building now...

Index: kernel/auditsc.c
===================================================================
--- b75da428ad7067959b24a0e2e6b64094750025b9/kernel/auditsc.c  (mode:100644)
+++ uncommitted/kernel/auditsc.c  (mode:100644)
@@ -557,7 +557,7 @@
 	rcu_read_unlock();
 	put_task_struct(tsk);
 
-	return 1; /* Audit by default */
+	return ret; /* Audit by default */
 
 }
 


-- 
dwmw2




More information about the Linux-audit mailing list