f assert that removed updates are actually ready to be removed

TheBlueMatt · TheBlueMatt · commit 70a0dff7ed31 · 2023-05-10T18:23:47.000Z
diff --git a/lightning/src/ln/channel.rs b/lightning/src/ln/channel.rs
@@ -5045,7 +5045,12 @@ impl<Signer: WriteableEcdsaChannelSigner> Channel<Signer> {
 	}
 
 	pub fn complete_all_mon_updates_through(&mut self, update_id: u64) {
-		self.pending_monitor_updates.retain(|upd| upd.update.update_id > update_id);
+		self.pending_monitor_updates.retain(|upd| {
+			if upd.update.update_id <= update_id {
+				assert!(!upd.blocked, "Completed update must have flown");
+				false
+			} else { true }
+		});
 	}
 
 	pub fn complete_one_mon_update(&mut self, update_id: u64) {

Original file line number	Diff line number	Diff line change
`@@ -5045,7 +5045,12 @@ impl<Signer: WriteableEcdsaChannelSigner> Channel<Signer> {`
`5045`	`5045`	`}`
`5046`	`5046`
`5047`	`5047`	`pub fn complete_all_mon_updates_through(&mut self, update_id: u64) {`
`5048`		`- self.pending_monitor_updates.retain(\|upd\| upd.update.update_id > update_id);`
	`5048`	`+ self.pending_monitor_updates.retain(\|upd\| {`
	`5049`	`+ if upd.update.update_id <= update_id {`
	`5050`	`+ assert!(!upd.blocked, "Completed update must have flown");`
	`5051`	`+ false`
	`5052`	`+ } else { true }`
	`5053`	`+ });`
`5049`	`5054`	`}`
`5050`	`5055`
`5051`	`5056`	`pub fn complete_one_mon_update(&mut self, update_id: u64) {`