fix: skip padding packets analogously to marker packets (#563)

hko-s · web-flow · commit d667b671bc83 · 2025-05-27T18:24:31.000+02:00
diff --git a/src/composed/shared.rs b/src/composed/shared.rs
@@ -215,14 +215,13 @@ pub trait Deserializable: Sized {
 
 /// Process results from low level packet parser:
 ///
-/// - Skip Marker packets.
+/// - Skip Marker and Padding packets.
 /// - Pass through other packets.
 /// - Skip any `Error::Unsupported`, those were marked as "safe to ignore" by the low level parser.
 /// - Skip `Error::Incomplete`
 /// - Skip `Error::EllipticCurve`
 /// - Pass through other errors.
 pub(crate) fn filter_parsed_packet_results(p: Result<Packet>) -> Option<Result<Packet>> {
-    // FIXME: handle padding packets (skip)
     // FIXME: handle criticality of packets from 9580 (error, if unsupported)
 
     match p {
@@ -231,6 +230,10 @@ pub(crate) fn filter_parsed_packet_results(p: Result<Packet>) -> Option<Result<P
                 debug!("skipping marker packet");
                 return None;
             }
+            if let Packet::Padding(_) = packet {
+                debug!("skipping padding packet");
+                return None;
+            }
             Some(p)
         }
         Err(e) => {
