.

Author: picnixz

Lib/test/test_sqlite3/test_dbapi.py

@@ -1079,10 +1079,11 @@ def __init__(self, cx, value):
                 self.value = value
 
             def __getitem__(self, i):
+                if self.value == 3:
+                    self.side_effect()
                 return self.value
 
             def __len__(self):
-                self.side_effect()
                 return 1
 
         class Ps(CxWrapper):

Modules/_sqlite/cursor.c

@@ -941,6 +941,9 @@ _pysqlite_query_execute(pysqlite_Cursor* self, int multiple, PyObject* operation
         }
 
         bind_parameters(state, self->statement, parameters);
+        // Note: bind_parameters() can have a side effect on the connection's
+        // state that could only be visible on the next API usage if we do not
+        // check it now.
         if (PyErr_Occurred() || !pysqlite_check_connection(self->connection)) {
             goto error;
         }