diff options
| author | Amy Griffis <amy.griffis@hp.com> | 2006-09-07 17:03:02 -0400 |
|---|---|---|
| committer | Al Viro <viro@zeniv.linux.org.uk> | 2006-09-11 13:32:17 -0400 |
| commit | 8ef2d3040e5cf38f7d64a408038f576b4a5ec987 (patch) | |
| tree | def11d400d2262e104cb1c64a953276794d8c9a8 /kernel | |
| parent | 3b33ac3182a4554742757a0c61ee1df162cf8225 (diff) | |
[PATCH] sanity check audit_buffer
Add sanity checks for NULL audit_buffer consistent with other
audit_log* routines.
Signed-off-by: Amy Griffis <amy.griffis@hp.com>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'kernel')
| -rw-r--r-- | kernel/audit.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/kernel/audit.c b/kernel/audit.c index 0a36091ed712..963fd15c9621 100644 --- a/kernel/audit.c +++ b/kernel/audit.c | |||
| @@ -1028,6 +1028,9 @@ void audit_log_hex(struct audit_buffer *ab, const unsigned char *buf, | |||
| 1028 | struct sk_buff *skb; | 1028 | struct sk_buff *skb; |
| 1029 | static const unsigned char *hex = "0123456789ABCDEF"; | 1029 | static const unsigned char *hex = "0123456789ABCDEF"; |
| 1030 | 1030 | ||
| 1031 | if (!ab) | ||
| 1032 | return; | ||
| 1033 | |||
| 1031 | BUG_ON(!ab->skb); | 1034 | BUG_ON(!ab->skb); |
| 1032 | skb = ab->skb; | 1035 | skb = ab->skb; |
| 1033 | avail = skb_tailroom(skb); | 1036 | avail = skb_tailroom(skb); |
| @@ -1060,6 +1063,9 @@ static void audit_log_n_string(struct audit_buffer *ab, size_t slen, | |||
| 1060 | unsigned char *ptr; | 1063 | unsigned char *ptr; |
| 1061 | struct sk_buff *skb; | 1064 | struct sk_buff *skb; |
| 1062 | 1065 | ||
| 1066 | if (!ab) | ||
| 1067 | return; | ||
| 1068 | |||
| 1063 | BUG_ON(!ab->skb); | 1069 | BUG_ON(!ab->skb); |
| 1064 | skb = ab->skb; | 1070 | skb = ab->skb; |
| 1065 | avail = skb_tailroom(skb); | 1071 | avail = skb_tailroom(skb); |
