FIX: Fix DbnFsm::consume fill logic

threecgreen · threecgreen · commit 31dbdece0020 · 2025-11-10T16:50:55.000Z
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -6,6 +6,9 @@
 - Added `MergeRecordDecoder::with_hints` that allows hinting the minimum timestamp from
   each decoder
 
+### Bug fixes
+- Fixed bug in `DbnFsm::consume`, though it should have no impact
+
 ## 0.43.0 - 2025-10-22
 
 ### Enhancements
diff --git a/rust/dbn/src/decode/dbn/fsm.rs b/rust/dbn/src/decode/dbn/fsm.rs
@@ -462,7 +462,7 @@ impl DbnFsm {
     fn consume(&mut self, read: usize, compat: usize, compat_fill: usize, expand_compat: bool) {
         self.buffer.consume(read);
         if compat_fill > 0 {
-            self.compat_buffer.fill(compat);
+            self.compat_buffer.fill(compat_fill);
         }
         if compat > 0 {
             self.compat_buffer.consume(compat);

Original file line number	Diff line number	Diff line change
`@@ -462,7 +462,7 @@ impl DbnFsm {`
`462`	`462`	`fn consume(&mut self, read: usize, compat: usize, compat_fill: usize, expand_compat: bool) {`
`463`	`463`	`self.buffer.consume(read);`
`464`	`464`	`if compat_fill > 0 {`
`465`		`- self.compat_buffer.fill(compat);`
	`465`	`+ self.compat_buffer.fill(compat_fill);`
`466`	`466`	`}`
`467`	`467`	`if compat > 0 {`
`468`	`468`	`self.compat_buffer.consume(compat);`