LSM: do not initialize common_audit_data to 0

It isn't needed.  If you don't set the type of the data associated with
that type it is a pretty obvious programming bug.  So why waste the cycles?

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

8 files changed, 8 insertions, 8 deletions

diff --git a/security/apparmor/capability.c b/security/apparmor/capability.c
index b66a0e4a5693..887a5e948945 100644
--- a/security/apparmor/capability.c
+++ b/security/apparmor/capability.c
@@ -65,7 +65,7 @@ static int audit_caps(struct aa_profile *profile, struct task_struct *task,
         int type = AUDIT_APPARMOR_AUTO;
         struct common_audit_data sa;
         struct apparmor_audit_data aad = {0,};
-        COMMON_AUDIT_DATA_INIT(&sa, LSM_AUDIT_DATA_CAP);
+        sa.type = LSM_AUDIT_DATA_CAP;
         sa.aad = &aad;
         sa.u.cap = cap;
         sa.aad->tsk = task;
diff --git a/security/apparmor/file.c b/security/apparmor/file.c
index 6ab264ca85cc..cf19d4093ca4 100644
--- a/security/apparmor/file.c
+++ b/security/apparmor/file.c
@@ -108,7 +108,7 @@ int aa_audit_file(struct aa_profile *profile, struct file_perms *perms,
         int type = AUDIT_APPARMOR_AUTO;
         struct common_audit_data sa;
         struct apparmor_audit_data aad = {0,};
-        COMMON_AUDIT_DATA_INIT(&sa, LSM_AUDIT_DATA_NONE);
+        sa.type = LSM_AUDIT_DATA_NONE;
         sa.aad = &aad;
         aad.op = op,
         aad.fs.request = request;
diff --git a/security/apparmor/ipc.c b/security/apparmor/ipc.c
index dba449b74db3..cf1071b14232 100644
--- a/security/apparmor/ipc.c
+++ b/security/apparmor/ipc.c
@@ -42,7 +42,7 @@ static int aa_audit_ptrace(struct aa_profile *profile,
 {
         struct common_audit_data sa;
         struct apparmor_audit_data aad = {0,};
-        COMMON_AUDIT_DATA_INIT(&sa, LSM_AUDIT_DATA_NONE);
+        sa.type = LSM_AUDIT_DATA_NONE;
         sa.aad = &aad;
         aad.op = OP_PTRACE;
         aad.target = target;
diff --git a/security/apparmor/lib.c b/security/apparmor/lib.c
index b11a2652f541..7430298116d6 100644
--- a/security/apparmor/lib.c
+++ b/security/apparmor/lib.c
@@ -66,7 +66,7 @@ void aa_info_message(const char *str)
         if (audit_enabled) {
                 struct common_audit_data sa;
                 struct apparmor_audit_data aad = {0,};
-                COMMON_AUDIT_DATA_INIT(&sa, LSM_AUDIT_DATA_NONE);
+                sa.type = LSM_AUDIT_DATA_NONE;
                 sa.aad = &aad;
                 aad.info = str;
                 aa_audit_msg(AUDIT_APPARMOR_STATUS, &sa, NULL);
diff --git a/security/apparmor/lsm.c b/security/apparmor/lsm.c
index 4f7bc07b2dce..032daab449b0 100644
--- a/security/apparmor/lsm.c
+++ b/security/apparmor/lsm.c
@@ -589,7 +589,7 @@ static int apparmor_setprocattr(struct task_struct *task, char *name,
                 } else {
                         struct common_audit_data sa;
                         struct apparmor_audit_data aad = {0,};
-                        COMMON_AUDIT_DATA_INIT(&sa, LSM_AUDIT_DATA_NONE);
+                        sa.type = LSM_AUDIT_DATA_NONE;
                         sa.aad = &aad;
                         aad.op = OP_SETPROCATTR;
                         aad.info = name;
diff --git a/security/apparmor/policy.c b/security/apparmor/policy.c
index 03dbaef2f8e3..421681c7c346 100644
--- a/security/apparmor/policy.c
+++ b/security/apparmor/policy.c
@@ -965,7 +965,7 @@ static int audit_policy(int op, gfp_t gfp, const char *name, const char *info,
 {
         struct common_audit_data sa;
         struct apparmor_audit_data aad = {0,};
-        COMMON_AUDIT_DATA_INIT(&sa, LSM_AUDIT_DATA_NONE);
+        sa.type = LSM_AUDIT_DATA_NONE;
         sa.aad = &aad;
         aad.op = op;
         aad.name = name;
diff --git a/security/apparmor/policy_unpack.c b/security/apparmor/policy_unpack.c
index 504ba4015aa2..329b1fd30749 100644
--- a/security/apparmor/policy_unpack.c
+++ b/security/apparmor/policy_unpack.c
@@ -95,7 +95,7 @@ static int audit_iface(struct aa_profile *new, const char *name,
         struct aa_profile *profile = __aa_current_profile();
         struct common_audit_data sa;
         struct apparmor_audit_data aad = {0,};
-        COMMON_AUDIT_DATA_INIT(&sa, LSM_AUDIT_DATA_NONE);
+        sa.type = LSM_AUDIT_DATA_NONE;
         sa.aad = &aad;
         if (e)
                 aad.iface.pos = e->pos - e->start;
diff --git a/security/apparmor/resource.c b/security/apparmor/resource.c
index d06f57b74f77..e1f3d7ef2c54 100644
--- a/security/apparmor/resource.c
+++ b/security/apparmor/resource.c
@@ -52,7 +52,7 @@ static int audit_resource(struct aa_profile *profile, unsigned int resource,
         struct common_audit_data sa;
         struct apparmor_audit_data aad = {0,};
 
-        COMMON_AUDIT_DATA_INIT(&sa, LSM_AUDIT_DATA_NONE);
+        sa.type = LSM_AUDIT_DATA_NONE;
         sa.aad = &aad;
         aad.op = OP_SETRLIMIT,
         aad.rlim.rlim = resource;