auditsc: Use kzalloc instead of kmalloc+memset.
authorRakib Mullick <rakib.mullick@gmail.com>
Sun, 7 Apr 2013 10:14:18 +0000 (16:14 +0600)
committerEric Paris <eparis@redhat.com>
Wed, 10 Apr 2013 19:18:24 +0000 (15:18 -0400)
  In function audit_alloc_context(), use kzalloc, instead of kmalloc+memset. Patch also renames audit_zero_context() to
audit_set_context(), to represent it's inner workings properly.

Signed-off-by: Rakib Mullick <rakib.mullick@gmail.com>
Signed-off-by: Eric Paris <eparis@redhat.com>
kernel/auditsc.c

index d57ad32db3676ee8431671b7ec6ccddd07268707..9dc3bae9793d27b4eb13d29561e1adecdd09d8e5 100644 (file)
@@ -1048,10 +1048,9 @@ static inline void audit_free_aux(struct audit_context *context)
        }
 }
 
-static inline void audit_zero_context(struct audit_context *context,
+static inline void audit_set_context(struct audit_context *context,
                                      enum audit_state state)
 {
-       memset(context, 0, sizeof(*context));
        context->state      = state;
        context->prio = state == AUDIT_RECORD_CONTEXT ? ~0ULL : 0;
 }
@@ -1060,9 +1059,10 @@ static inline struct audit_context *audit_alloc_context(enum audit_state state)
 {
        struct audit_context *context;
 
-       if (!(context = kmalloc(sizeof(*context), GFP_KERNEL)))
+       context = kzalloc(sizeof(*context), GFP_KERNEL);
+       if (!context)
                return NULL;
-       audit_zero_context(context, state);
+       audit_set_context(context, state);
        INIT_LIST_HEAD(&context->killed_trees);
        INIT_LIST_HEAD(&context->names_list);
        return context;