diff options
author | Ulrich Drepper <drepper@redhat.com> | 2008-07-24 00:29:42 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2008-07-24 13:47:29 -0400 |
commit | e38b36f325153eaadd1c2a7abc5762079233e540 (patch) | |
tree | 92cfc9855e41c5328d91456f5e373c00ecb8d383 /net/socket.c | |
parent | 510df2dd482496083e1c3b1a8c9b6afd5fa4c7d7 (diff) |
flag parameters: check magic constants
This patch adds test that ensure the boundary conditions for the various
constants introduced in the previous patches is met. No code is generated.
[akpm@linux-foundation.org: fix alpha]
Signed-off-by: Ulrich Drepper <drepper@redhat.com>
Acked-by: Davide Libenzi <davidel@xmailserver.org>
Cc: Michael Kerrisk <mtk.manpages@googlemail.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'net/socket.c')
-rw-r--r-- | net/socket.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/net/socket.c b/net/socket.c index 31105f9048a8..1310a82cbba7 100644 --- a/net/socket.c +++ b/net/socket.c | |||
@@ -1222,6 +1222,12 @@ asmlinkage long sys_socket(int family, int type, int protocol) | |||
1222 | struct socket *sock; | 1222 | struct socket *sock; |
1223 | int flags; | 1223 | int flags; |
1224 | 1224 | ||
1225 | /* Check the SOCK_* constants for consistency. */ | ||
1226 | BUILD_BUG_ON(SOCK_CLOEXEC != O_CLOEXEC); | ||
1227 | BUILD_BUG_ON((SOCK_MAX | SOCK_TYPE_MASK) != SOCK_TYPE_MASK); | ||
1228 | BUILD_BUG_ON(SOCK_CLOEXEC & SOCK_TYPE_MASK); | ||
1229 | BUILD_BUG_ON(SOCK_NONBLOCK & SOCK_TYPE_MASK); | ||
1230 | |||
1225 | flags = type & ~SOCK_TYPE_MASK; | 1231 | flags = type & ~SOCK_TYPE_MASK; |
1226 | if (flags & ~(SOCK_CLOEXEC | SOCK_NONBLOCK)) | 1232 | if (flags & ~(SOCK_CLOEXEC | SOCK_NONBLOCK)) |
1227 | return -EINVAL; | 1233 | return -EINVAL; |