]> git.bitcoin.ninja Git - flowspec-xdp/commitdiff
Correct second-frag L4 matching
authorMatt Corallo <git@bluematt.me>
Sun, 4 Apr 2021 17:14:08 +0000 (13:14 -0400)
committerMatt Corallo <git@bluematt.me>
Sun, 4 Apr 2021 17:14:08 +0000 (13:14 -0400)
test.sh
xdp.c

diff --git a/test.sh b/test.sh
index 10ae086891a5f604a1dfb20c6e7807f6ecf15216..f13d3f9df1dbdedb525a5f1a5db586cefb44c2f6 100755 (executable)
--- a/test.sh
+++ b/test.sh
@@ -228,6 +228,17 @@ echo "$TEST_PKT" >> rules.h
 echo "#define TEST_EXP XDP_PASS" >> rules.h
 clang -std=c99 -fsanitize=address -pedantic -Wall -Wextra -Wno-pointer-arith -Wno-unused-variable -O0 -g xdp.c -o xdp && ./xdp
 
+echo "flow6 { src 2620:6e:a007:233::1/128; dst 2001:470:0:503::2/128; fragment !is_fragment || first_fragment || !last_fragment; };" | ./genrules.py --ihl=drop-options --8021q=drop-vlan --v6frag=parse-frags
+echo "$TEST_PKT" >> rules.h
+echo "#define TEST_EXP XDP_PASS" >> rules.h
+clang -std=c99 -fsanitize=address -pedantic -Wall -Wextra -Wno-pointer-arith -Wno-unused-variable -O0 -g xdp.c -o xdp && ./xdp
+
+# Note on the second fragment we don't know the ICMP header (though < 256 is trivially true)
+echo "flow6 { icmp type < 256; };" | ./genrules.py --ihl=drop-options --8021q=drop-vlan --v6frag=parse-frags
+echo "$TEST_PKT" >> rules.h
+echo "#define TEST_EXP XDP_PASS" >> rules.h
+clang -std=c99 -fsanitize=address -pedantic -Wall -Wextra -Wno-pointer-arith -Wno-unused-variable -O0 -g xdp.c -o xdp && ./xdp
+
 #TODO Is nextheader frag correct to match on here? Should we support matching on any nexthdr?
 echo "flow6 { next header 44; };" | ./genrules.py --ihl=accept-options --8021q=accept-vlan --v6frag=parse-frags
 echo "$TEST_PKT" >> rules.h
diff --git a/xdp.c b/xdp.c
index 1709e00cfb727e308c541d60441c8ce50c828707..74e8e8cb0e850c8556b68877c0faa6de194f51b4 100644 (file)
--- a/xdp.c
+++ b/xdp.c
@@ -226,18 +226,20 @@ int xdp_drop_prog(struct xdp_md *ctx)
                l4hdr = pktdata + 5*4;
 #endif
 
-               if (ip->protocol == IP_PROTO_TCP) {
-                       if (unlikely(l4hdr + sizeof(struct tcphdr) > data_end))
-                               DO_RETURN(PKT_LEN_DROP, XDP_DROP);
-                       tcp = (struct tcphdr*) l4hdr;
-               } else if (ip->protocol == IP_PROTO_UDP) {
-                       if (unlikely(l4hdr + sizeof(struct udphdr) > data_end))
-                               DO_RETURN(PKT_LEN_DROP, XDP_DROP);
-                       udp = (struct udphdr*) l4hdr;
-               } else if (ip->protocol == IP_PROTO_ICMP) {
-                       if (unlikely(l4hdr + sizeof(struct icmphdr) > data_end))
-                               DO_RETURN(PKT_LEN_DROP, XDP_DROP);
-                       icmp = (struct icmphdr*) l4hdr;
+               if ((ip->frag_off & BE16(IP_OFFSET)) == 0) {
+                       if (ip->protocol == IP_PROTO_TCP) {
+                               if (unlikely(l4hdr + sizeof(struct tcphdr) > data_end))
+                                       DO_RETURN(PKT_LEN_DROP, XDP_DROP);
+                               tcp = (struct tcphdr*) l4hdr;
+                       } else if (ip->protocol == IP_PROTO_UDP) {
+                               if (unlikely(l4hdr + sizeof(struct udphdr) > data_end))
+                                       DO_RETURN(PKT_LEN_DROP, XDP_DROP);
+                               udp = (struct udphdr*) l4hdr;
+                       } else if (ip->protocol == IP_PROTO_ICMP) {
+                               if (unlikely(l4hdr + sizeof(struct icmphdr) > data_end))
+                                       DO_RETURN(PKT_LEN_DROP, XDP_DROP);
+                               icmp = (struct icmphdr*) l4hdr;
+                       }
                }
        }
 #endif
@@ -264,21 +266,23 @@ int xdp_drop_prog(struct xdp_md *ctx)
 #endif
                }
 #endif
-
-               if (v6nexthdr == IP_PROTO_TCP) {
-                       if (unlikely(l4hdr + sizeof(struct tcphdr) > data_end))
-                               DO_RETURN(PKT_LEN_DROP, XDP_DROP);
-                       tcp = (struct tcphdr*) l4hdr;
-               } else if (v6nexthdr == IP_PROTO_UDP) {
-                       if (unlikely(l4hdr + sizeof(struct udphdr) > data_end))
-                               DO_RETURN(PKT_LEN_DROP, XDP_DROP);
-                       udp = (struct udphdr*) l4hdr;
-               } else if (v6nexthdr == IP6_PROTO_ICMPV6) {
-                       if (unlikely(l4hdr + sizeof(struct icmp6hdr) > data_end))
-                               DO_RETURN(PKT_LEN_DROP, XDP_DROP);
-                       icmpv6 = (struct icmp6hdr*) l4hdr;
+               // TODO: Handle more options?
+
+               if (frag6 == NULL || (frag6->frag_off & BE16(IP6_FRAGOFF)) == 0) {
+                       if (v6nexthdr == IP_PROTO_TCP) {
+                               if (unlikely(l4hdr + sizeof(struct tcphdr) > data_end))
+                                       DO_RETURN(PKT_LEN_DROP, XDP_DROP);
+                               tcp = (struct tcphdr*) l4hdr;
+                       } else if (v6nexthdr == IP_PROTO_UDP) {
+                               if (unlikely(l4hdr + sizeof(struct udphdr) > data_end))
+                                       DO_RETURN(PKT_LEN_DROP, XDP_DROP);
+                               udp = (struct udphdr*) l4hdr;
+                       } else if (v6nexthdr == IP6_PROTO_ICMPV6) {
+                               if (unlikely(l4hdr + sizeof(struct icmp6hdr) > data_end))
+                                       DO_RETURN(PKT_LEN_DROP, XDP_DROP);
+                               icmpv6 = (struct icmp6hdr*) l4hdr;
+                       }
                }
-               // TODO: Handle some options?
        }
 #endif