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:03 -0400 |
commit | 07f62eb66c6626aa5653a0fcb34c9c040d0bd032 (patch) | |
tree | a928c034e2f08ef3f7fd6af450d75f3d52a261db /security/lsm_audit.c | |
parent | b466066f9b648ccb6aa1e174f0389b7433e460fd (diff) |
LSM: BUILD_BUG_ON if the common_audit_data union ever grows
We did a lot of work to shrink the common_audit_data. Add a BUILD_BUG_ON
so future programers (let's be honest, probably me) won't do something
foolish like make it large again!
Signed-off-by: Eric Paris <eparis@redhat.com>
Diffstat (limited to 'security/lsm_audit.c')
-rw-r--r-- | security/lsm_audit.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/security/lsm_audit.c b/security/lsm_audit.c index e796d2517653..8d8d97dbb389 100644 --- a/security/lsm_audit.c +++ b/security/lsm_audit.c | |||
@@ -213,6 +213,13 @@ static void dump_common_audit_data(struct audit_buffer *ab, | |||
213 | { | 213 | { |
214 | struct task_struct *tsk = current; | 214 | struct task_struct *tsk = current; |
215 | 215 | ||
216 | /* | ||
217 | * To keep stack sizes in check force programers to notice if they | ||
218 | * start making this union too large! See struct lsm_network_audit | ||
219 | * as an example of how to deal with large data. | ||
220 | */ | ||
221 | BUILD_BUG_ON(sizeof(a->u) > sizeof(void *)*2); | ||
222 | |||
216 | audit_log_format(ab, " pid=%d comm=", tsk->pid); | 223 | audit_log_format(ab, " pid=%d comm=", tsk->pid); |
217 | audit_log_untrustedstring(ab, tsk->comm); | 224 | audit_log_untrustedstring(ab, tsk->comm); |
218 | 225 | ||