aboutsummaryrefslogtreecommitdiffstats
path: root/kernel/audit.c
diff options
context:
space:
mode:
authorPaul Moore <paul@paul-moore.com>2018-08-02 17:56:50 -0400
committerPaul Moore <paul@paul-moore.com>2018-11-26 18:40:00 -0500
commitd0a3f18a70f2d9700bf9f5e9c4a505472388a7c1 (patch)
treeb316add3dac38538bff1756da5d4c8c4f2fd2950 /kernel/audit.c
parentc8fc5d49c341805fee7fc295f2ea8a709f78aec4 (diff)
audit: minimize our use of audit_log_format()
There are some cases where we are making multiple audit_log_format() calls in a row, for no apparent reason. Squash these down to a single audit_log_format() call whenever possible. Acked-by: Richard Guy Briggs <rgb@redhat.com> Signed-off-by: Paul Moore <paul@paul-moore.com>
Diffstat (limited to 'kernel/audit.c')
-rw-r--r--kernel/audit.c11
1 files changed, 5 insertions, 6 deletions
diff --git a/kernel/audit.c b/kernel/audit.c
index 6c53e373b828..d09298d3c2d2 100644
--- a/kernel/audit.c
+++ b/kernel/audit.c
@@ -2177,22 +2177,21 @@ void audit_log_name(struct audit_context *context, struct audit_names *n,
2177 } 2177 }
2178 2178
2179 /* log the audit_names record type */ 2179 /* log the audit_names record type */
2180 audit_log_format(ab, " nametype=");
2181 switch(n->type) { 2180 switch(n->type) {
2182 case AUDIT_TYPE_NORMAL: 2181 case AUDIT_TYPE_NORMAL:
2183 audit_log_format(ab, "NORMAL"); 2182 audit_log_format(ab, " nametype=NORMAL");
2184 break; 2183 break;
2185 case AUDIT_TYPE_PARENT: 2184 case AUDIT_TYPE_PARENT:
2186 audit_log_format(ab, "PARENT"); 2185 audit_log_format(ab, " nametype=PARENT");
2187 break; 2186 break;
2188 case AUDIT_TYPE_CHILD_DELETE: 2187 case AUDIT_TYPE_CHILD_DELETE:
2189 audit_log_format(ab, "DELETE"); 2188 audit_log_format(ab, " nametype=DELETE");
2190 break; 2189 break;
2191 case AUDIT_TYPE_CHILD_CREATE: 2190 case AUDIT_TYPE_CHILD_CREATE:
2192 audit_log_format(ab, "CREATE"); 2191 audit_log_format(ab, " nametype=CREATE");
2193 break; 2192 break;
2194 default: 2193 default:
2195 audit_log_format(ab, "UNKNOWN"); 2194 audit_log_format(ab, " nametype=UNKNOWN");
2196 break; 2195 break;
2197 } 2196 }
2198 2197