From e9f47da5c530abc942bd05893581187a08271f49 Mon Sep 17 00:00:00 2001
From: Yasuhiro Matsumoto <mattn.jp@gmail.com>
Date: Wed, 8 Apr 2026 13:40:21 +0900
Subject: [PATCH] do not bail out on finalize error in closeCachedStmtsLocked

Finalize all cached statements even if one fails. Leaving a
finalized statement in the cache map would be a use-after-finalize
bug per SQLite documentation.
---
 sqlite3.go | 11 +++--------
 1 file changed, 3 insertions(+), 8 deletions(-)

diff --git a/sqlite3.go b/sqlite3.go
index a05cfd4..d634d6f 100644
--- a/sqlite3.go
+++ b/sqlite3.go
@@ -1884,9 +1884,7 @@ func (c *SQLiteConn) Close() error {
 		return nil
 	}
 	runtime.SetFinalizer(c, nil)
-	if err := c.closeCachedStmtsLocked(); err != nil {
-		return err
-	}
+	c.closeCachedStmtsLocked()
 	rv := C.sqlite3_close_v2(c.db)
 	if rv != C.SQLITE_OK {
 		return lastError(c.db)
@@ -1949,23 +1947,20 @@ func (c *SQLiteConn) putCachedStmt(s *SQLiteStmt) bool {
 	return true
 }
 
-func (c *SQLiteConn) closeCachedStmtsLocked() error {
+func (c *SQLiteConn) closeCachedStmtsLocked() {
 	for key, stmts := range c.stmtCache {
 		for _, s := range stmts {
 			if s == nil || s.s == nil {
 				continue
 			}
 			runtime.SetFinalizer(s, nil)
-			if rv := C.sqlite3_finalize(s.s); rv != C.SQLITE_OK {
-				return lastError(c.db)
-			}
+			C.sqlite3_finalize(s.s)
 			s.s = nil
 			s.c = nil
 		}
 		delete(c.stmtCache, key)
 	}
 	c.stmtCacheCount = 0
-	return nil
 }
 
 // Prepare the query string. Return a new statement.