summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDaniel Borkmann <dborkman@redhat.com>2014-05-29 10:22:49 +0200
committerDavid S. Miller <davem@davemloft.net>2014-06-01 22:16:58 -0700
commitd50bc1575096250aa37f17299c86ea548156efe8 (patch)
treef606b52b3a98a9a3a9514dcf774cdff0b83b3499
parent9fe13baad6b1e8d3063dd48faf1b9be9b7c5e92c (diff)
downloadblackbird-op-linux-d50bc1575096250aa37f17299c86ea548156efe8.tar.gz
blackbird-op-linux-d50bc1575096250aa37f17299c86ea548156efe8.zip
net: filter: add test for loading SKF_AD_OFF limits
This check tests that overloading BPF_LD | BPF_ABS with an always invalid BPF extension, that is SKF_AD_MAX, fails to make sure classic BPF behaviour is correct in filter checker. Also, we add a test for loading at packet offset SKF_AD_OFF-1 which should pass the filter, but later on fail during runtime. Signed-off-by: Daniel Borkmann <dborkman@redhat.com> Acked-by: Alexei Starovoitov <ast@plumgrid.com> Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r--lib/test_bpf.c22
1 files changed, 22 insertions, 0 deletions
diff --git a/lib/test_bpf.c b/lib/test_bpf.c
index 2d0a0d141793..f8d2b2a13131 100644
--- a/lib/test_bpf.c
+++ b/lib/test_bpf.c
@@ -1655,6 +1655,28 @@ static struct bpf_test tests[] = {
{ },
{ { 0, 0x2a5a5e5 } },
},
+ {
+ "check: SKF_AD_MAX",
+ .u.insns = {
+ BPF_STMT(BPF_LD | BPF_W | BPF_ABS,
+ SKF_AD_OFF + SKF_AD_MAX),
+ BPF_STMT(BPF_RET | BPF_A, 0),
+ },
+ CLASSIC | FLAG_NO_DATA | FLAG_EXPECTED_FAIL,
+ { },
+ { },
+ },
+ { /* Passes checker but fails during runtime. */
+ "LD [SKF_AD_OFF-1]",
+ .u.insns = {
+ BPF_STMT(BPF_LD | BPF_W | BPF_ABS,
+ SKF_AD_OFF - 1),
+ BPF_STMT(BPF_RET | BPF_K, 1),
+ },
+ CLASSIC,
+ { },
+ { { 1, 0 } },
+ },
};
static struct net_device dev;
OpenPOWER on IntegriCloud