diff options
| author | Eric Paris <eparis@redhat.com> | 2013-11-05 10:48:02 -0500 |
|---|---|---|
| committer | Eric Paris <eparis@redhat.com> | 2013-11-05 11:08:35 -0500 |
| commit | e13f91e3c57986a609c10ddf94af0546a2a97dce (patch) | |
| tree | 71c58b8eaa95bcffb554052794f3c99e767104da /kernel/audit.c | |
| parent | 64fbff9ae0a0a843365d922e0057fc785f23f0e3 (diff) | |
| download | op-kernel-dev-e13f91e3c57986a609c10ddf94af0546a2a97dce.zip op-kernel-dev-e13f91e3c57986a609c10ddf94af0546a2a97dce.tar.gz | |
audit: use memset instead of trying to initialize field by field
We currently are setting fields to 0 to initialize the structure
declared on the stack. This is a bad idea as if the structure has holes
or unpacked space these will not be initialized. Just use memset. This
is not a performance critical section of code.
Signed-off-by: Eric Paris <eparis@redhat.com>
Diffstat (limited to 'kernel/audit.c')
| -rw-r--r-- | kernel/audit.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/audit.c b/kernel/audit.c index e0f7767..3ce60e0 100644 --- a/kernel/audit.c +++ b/kernel/audit.c @@ -759,7 +759,7 @@ static int audit_receive_msg(struct sk_buff *skb, struct nlmsghdr *nlh) switch (msg_type) { case AUDIT_GET: - status_set.mask = 0; + memset(&status_set, 0, sizeof(status_set)); status_set.enabled = audit_enabled; status_set.failure = audit_failure; status_set.pid = audit_pid; |
