Merge pull request #5148 from oasisprotocol/kostko/fix/clear-verifica…

…tion-trace runtime: Clear verification trace after block verification
oasisprotocol · Jan 18, 2023 · 700e5dc · 700e5dc
2 parents 473dade + f107cf3
commit 700e5dc
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/.changelog/5148.bugfix.md b/.changelog/5148.bugfix.md
@@ -0,0 +1 @@
+runtime: Clear verification trace after block verification
diff --git a/runtime/src/consensus/tendermint/verifier/mod.rs b/runtime/src/consensus/tendermint/verifier/mod.rs
@@ -143,6 +143,9 @@ impl Verifier {
         }
         .map_err(|err| Error::VerificationFailed(err.into()))?;
 
+        // Clear verification trace as it could otherwise lead to infinite memory growth.
+        instance.state.verification_trace.clear();
+
         cache.update_verified_block(&verified_block);
         self.update_insecure_posix_time(&verified_block);
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		runtime: Clear verification trace after block verification