Merge pull request #3322 from autonomys/cross-segment-object-bug

Fix bug in cross-segment object retrieval
autonomys · Dec 18, 2024 · 778d965 · 778d965
2 parents 3d5483e + 1e579b3
commit 778d965
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/shared/subspace-data-retrieval/src/object_fetcher.rs b/shared/subspace-data-retrieval/src/object_fetcher.rs
@@ -514,13 +514,12 @@ where
                         }
                     }
 
-                    // Padding at the end of segments can be skipped, it's not part of the object data
-                    SegmentItem::Padding => {}
+                    // Padding at the end of segments, and segment headers can be skipped,
+                    // they're not part of the object data
+                    SegmentItem::Padding | SegmentItem::ParentSegmentHeader(_) => {}
 
                     // We should not see these items while collecting data for a single object
-                    SegmentItem::Block { .. }
-                    | SegmentItem::BlockStart { .. }
-                    | SegmentItem::ParentSegmentHeader(_) => {
+                    SegmentItem::Block { .. } | SegmentItem::BlockStart { .. } => {
                         debug!(
                             collected_data = ?data.len(),
                             %segment_index,