Audit: clean up all op= output to include string quoting

A number of places in the audit system we send an op= followed by a string
that includes spaces.  Somehow this works but it's just wrong.  This patch
moves all of those that I could find to be quoted.

Example:

Change From: type=CONFIG_CHANGE msg=audit(1244666690.117:31): auid=0 ses=1
subj=unconfined_u:unconfined_r:auditctl_t:s0-s0:c0.c1023 op=remove rule
key="number2" list=4 res=0

Change To: type=CONFIG_CHANGE msg=audit(1244666690.117:31): auid=0 ses=1
subj=unconfined_u:unconfined_r:auditctl_t:s0-s0:c0.c1023 op="remove rule"
key="number2" list=4 res=0

Signed-off-by: Eric Paris <eparis@redhat.com>

1 files changed, 1 insertions, 5 deletions

diff --git a/kernel/audit_watch.c b/kernel/audit_watch.c
index b49ab019fdff..0e96dbc60ea9 100644
--- a/kernel/audit_watch.c
+++ b/kernel/audit_watch.c
@@ -234,11 +234,7 @@ static void audit_watch_log_rule_change(struct audit_krule *r, struct audit_watc
                 audit_log_string(ab, op);
                 audit_log_format(ab, " path=");
                 audit_log_untrustedstring(ab, w->path);
-                if (r->filterkey) {
-                        audit_log_format(ab, " key=");
-                        audit_log_untrustedstring(ab, r->filterkey);
-                } else
-                        audit_log_format(ab, " key=(null)");
+                audit_log_key(ab, r->filterkey);
                 audit_log_format(ab, " list=%d res=1", r->listnr);
                 audit_log_end(ab);
         }