diff options
| author | Eric Paris <eparis@redhat.com> | 2012-04-04 21:01:43 +0200 |
|---|---|---|
| committer | Eric Paris <eparis@redhat.com> | 2012-04-09 18:23:04 +0200 |
| commit | 50c205f5e5c2e2af002fd4ef537ded79b90b1b56 (patch) | |
| tree | 9965a7746aa8c5e982357d5b8c46850f3283206c /security/smack | |
| parent | LSM: BUILD_BUG_ON if the common_audit_data union ever grows (diff) | |
| download | linux-50c205f5e5c2e2af002fd4ef537ded79b90b1b56.tar.xz linux-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/smack')
| -rw-r--r-- | security/smack/smack.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/security/smack/smack.h b/security/smack/smack.h index 4ede719922ed..b61e75f224d4 100644 --- a/security/smack/smack.h +++ b/security/smack/smack.h @@ -319,7 +319,7 @@ void smack_log(char *subject_label, char *object_label, static inline void smk_ad_init(struct smk_audit_info *a, const char *func, char type) { - memset(a, 0, sizeof(*a)); + memset(&a->sad, 0, sizeof(a->sad)); a->a.type = type; a->a.smack_audit_data = &a->sad; a->a.smack_audit_data->function = func; |