diff options
author | David Ahern <dsahern@gmail.com> | 2017-09-01 18:18:07 +0300 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2017-09-01 20:37:14 +0300 |
commit | 9df59055ed8e2f817149e786b2a52bc17832d84e (patch) | |
tree | c70aa33ec7c7f4c84e0d64f15a3a8745f49e6dde /net/core/filter.c | |
parent | c54a5048904b6a492551d1cf0a7c9ae3ca763f4b (diff) | |
download | linux-9df59055ed8e2f817149e786b2a52bc17832d84e.tar.xz |
bpf: Collapse offset checks in sock_filter_is_valid_access
Make sock_filter_is_valid_access consistent with other is_valid_access
helpers.
Requested-by: Daniel Borkmann <daniel@iogearbox.net>
Signed-off-by: David Ahern <dsahern@gmail.com>
Acked-by: Daniel Borkmann <daniel@iogearbox.net>
Acked-by: Alexei Starovoitov <ast@kernel.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/core/filter.c')
-rw-r--r-- | net/core/filter.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/net/core/filter.c b/net/core/filter.c index 9dad3e7e2e10..f9add024d92f 100644 --- a/net/core/filter.c +++ b/net/core/filter.c @@ -3468,9 +3468,7 @@ static bool sock_filter_is_valid_access(int off, int size, if (type == BPF_WRITE) { switch (off) { case offsetof(struct bpf_sock, bound_dev_if): - break; case offsetof(struct bpf_sock, mark): - break; case offsetof(struct bpf_sock, priority): break; default: |