On Thursday 08 November 2007 09:19:26 Alexander Viro wrote:
Have fun...
Thanks for posting this patch. Is it impossible to "repair " processes by
simply adding a context if the pointer is NULL?
diff --git a/kernel/audit.c b/kernel/audit.c
index f93c271..83227f8 100644
--- a/kernel/audit.c
+++ b/kernel/audit.c
@@ -70,6 +70,7 @@ static int audit_initialized;
* 1 - auditing enabled
* 2 - auditing enabled and configuration is locked/unchangeable. */
int audit_enabled;
+int audit_ever_enabled;
/* Default state when kernel boots without any parameters. */
static int audit_default;
@@ -965,6 +968,7 @@ static int __init audit_init(void)
skb_queue_head_init(&audit_skb_queue);
audit_initialized = 1;
audit_enabled = audit_default;
+ audit_ever_enabled |= !!audit_default;
Should the declaration of audit_ever_enabled set a default value like 0 since
this is being or'ed in? Or should this just be an assignment?
-Steve