diff options
| author |   Dan Carpenter <dan.carpenter@oracle.com> | 2011-12-07 20:49:38 +0000 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2011-12-08 19:58:35 -0500 |
| commit | 6f8e4ad0eff5117ba895122674670f9c63b6e8d8 (patch) | |
| tree | b84f9c6566a1d6b178b8ca2d021919b5c3499367 /net/core | |
| parent | net: netprio_cgroup: make net_prio_subsys static (diff) | |
| download | linux-dev-6f8e4ad0eff5117ba895122674670f9c63b6e8d8.tar.xz linux-dev-6f8e4ad0eff5117ba895122674670f9c63b6e8d8.zip | |
sock_diag: off by one checks
These tests are off by one because sock_diag_handlers[] only has AF_MAX
elements.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Pavel Emelyanov <xemul@parallels.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/core')
| -rw-r--r-- | net/core/sock_diag.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/core/sock_diag.c b/net/core/sock_diag.c index fbaf01ce12f2..9c27bcda59be 100644 --- a/net/core/sock_diag.c +++ b/net/core/sock_diag.c @@ -32,7 +32,7 @@ int sock_diag_register(struct sock_diag_handler *hndl) { int err = 0; - if (hndl->family > AF_MAX) + if (hndl->family >= AF_MAX) return -EINVAL; mutex_lock(&sock_diag_table_mutex); @@ -50,7 +50,7 @@ void sock_diag_unregister(struct sock_diag_handler *hnld) { int family = hnld->family; - if (family > AF_MAX) + if (family >= AF_MAX) return; mutex_lock(&sock_diag_table_mutex); |