]> git.bitcoin.ninja Git - dnssec-prover/commitdiff
Rm debug assertions that `read_wire_packet_name` empties its buffer
authorMatt Corallo <git@bluematt.me>
Tue, 9 Jul 2024 21:08:19 +0000 (21:08 +0000)
committerMatt Corallo <git@bluematt.me>
Tue, 9 Jul 2024 21:11:12 +0000 (21:11 +0000)
These assertions are spurious and reachable when fuzzing, and thus
are simply removed.

src/rr.rs

index 0733dd2fa2897d79836a5632e4d5c9d33d7df93b..a3839d29bf317c71e9260a1331c4e21e6055cfd1 100644 (file)
--- a/src/rr.rs
+++ b/src/rr.rs
@@ -401,7 +401,6 @@ impl StaticRecord for CName {
        }
        fn read_from_data(name: Name, mut data: &[u8], wire_packet: &[u8]) -> Result<Self, ()> {
                let res = CName { name, canonical_name: read_wire_packet_name(&mut data, wire_packet)? };
-               debug_assert!(data.is_empty());
                Ok(res)
        }
        fn write_u16_len_prefixed_data<W: Writer>(&self, out: &mut W) {
@@ -432,7 +431,6 @@ impl StaticRecord for DName {
        }
        fn read_from_data(name: Name, mut data: &[u8], wire_packet: &[u8]) -> Result<Self, ()> {
                let res = DName { name, delegation_name: read_wire_packet_name(&mut data, wire_packet)? };
-               debug_assert!(data.is_empty());
                Ok(res)
        }
        fn write_u16_len_prefixed_data<W: Writer>(&self, out: &mut W) {
@@ -877,7 +875,6 @@ impl StaticRecord for NS {
        }
        fn read_from_data(name: Name, mut data: &[u8], wire_packet: &[u8]) -> Result<Self, ()> {
                let res = NS { name, name_server: read_wire_packet_name(&mut data, wire_packet)? };
-               debug_assert!(data.is_empty());
                Ok(res)
        }
        fn write_u16_len_prefixed_data<W: Writer>(&self, out: &mut W) {