From de142957b35dc9f48c7d0fda274e7a78b3c6e565 Mon Sep 17 00:00:00 2001
From: Daniel Liu <139250065@qq.com>
Date: Wed, 11 Mar 2026 09:56:37 +0800
Subject: [PATCH] refactor(core/txpool): don't inject lazy resolved
 transactions into the container #28917 (#2133)

---
 core/txpool/subpool.go | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/core/txpool/subpool.go b/core/txpool/subpool.go
index cfc038362d..c20115ff0e 100644
--- a/core/txpool/subpool.go
+++ b/core/txpool/subpool.go
@@ -43,11 +43,17 @@ type LazyTransaction struct {
 
 // Resolve retrieves the full transaction belonging to a lazy handle if it is still
 // maintained by the transaction pool.
+//
+// Note, the method will *not* cache the retrieved transaction if the original
+// pool has not cached it. The idea being, that if the tx was too big to insert
+// originally, silently saving it will cause more trouble down the line (and
+// indeed seems to have caused a memory bloat in the original implementation
+// which did just that).
 func (ltx *LazyTransaction) Resolve() *types.Transaction {
-	if ltx.Tx == nil {
-		ltx.Tx = ltx.Pool.Get(ltx.Hash)
+	if ltx.Tx != nil {
+		return ltx.Tx
 	}
-	return ltx.Tx
+	return ltx.Pool.Get(ltx.Hash)
 }
 
 // LazyResolver is a minimal interface needed for a transaction pool to satisfy