diff options
author | Paul Moore <paul@paul-moore.com> | 2018-08-02 17:56:50 -0400 |
---|---|---|
committer | Paul Moore <paul@paul-moore.com> | 2018-11-26 18:40:00 -0500 |
commit | d0a3f18a70f2d9700bf9f5e9c4a505472388a7c1 (patch) | |
tree | b316add3dac38538bff1756da5d4c8c4f2fd2950 /kernel/audit.c | |
parent | c8fc5d49c341805fee7fc295f2ea8a709f78aec4 (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.c | 11 |
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 | ||