diff options
| author |   Eric Paris <eparis@redhat.com> | 2012-04-04 15:01:43 -0400 |
|---|---|---|
| committer | Eric Paris <eparis@redhat.com> | 2012-04-09 12:23:04 -0400 |
| commit | 50c205f5e5c2e2af002fd4ef537ded79b90b1b56 (patch) | |
| tree | 9965a7746aa8c5e982357d5b8c46850f3283206c /security/apparmor/lsm.c | |
| parent | LSM: BUILD_BUG_ON if the common_audit_data union ever grows (diff) | |
| download | linux-dev-50c205f5e5c2e2af002fd4ef537ded79b90b1b56.tar.xz linux-dev-50c205f5e5c2e2af002fd4ef537ded79b90b1b56.zip | |
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>
Diffstat (limited to 'security/apparmor/lsm.c')
| -rw-r--r-- | security/apparmor/lsm.c | 2 |
1 files changed, 1 insertions, 1 deletions
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; |