Merge #534

534: Fix assert with any_ip + broadcast dst_addr. r=Dirbaio a=Dirbaio

Fixes #533

Co-authored-by: Dario Nieuwenhuis <dirbaio@dirbaio.net>

.github/workflows/fuzz.yml

@@ -16,6 +16,9 @@ jobs:
           toolchain: nightly
           override: true
       - name: Install cargo-fuzz
-        run: cargo install cargo-fuzz
+        # Fix for cargo-fuzz on latest nightly: https://github.com/rust-fuzz/cargo-fuzz/issues/276
+        # Switch back to installing from crates.io when it's released.
+        #run: cargo install cargo-fuzz
+        run: cargo install --git https://github.com/rust-fuzz/cargo-fuzz --rev b4df3e58f767b5cad8d1aa6753961003f56f3609
       - name: Fuzz
         run: cargo fuzz run packet_parser -- -max_len=1536 -max_total_time=30

src/iface/interface.rs

@@ -1239,6 +1239,7 @@ impl<'a> InterfaceInner<'a> {
             // Ignore IP packets not directed at us, or broadcast, or any of the multicast groups.
             // If AnyIP is enabled, also check if the packet is routed locally.
             if !self.any_ip
+                || !ipv4_repr.dst_addr.is_unicast()
                 || self
                     .routes
                     .lookup(&IpAddress::Ipv4(ipv4_repr.dst_addr), cx.now)