Error "DPY-3027: binding a cursor from a different connection is not

supported" is now raised when attempting to bind a cursor created on a
different connection. Previously, the attempt may have succeeded or may
have failed with a number of different unexpected exceptions.

Author: anthony-tuininga

doc/src/release_notes.rst

@@ -31,6 +31,10 @@ Thick Mode Changes
 Common Changes
 ++++++++++++++
 
+#)  Error ``DPY-3027: binding a cursor from a different connection is not
+    supported`` is now raised when attempting to bind a cursor created on a
+    different connection. Previously, the attempt may have succeeded or may
+    have failed with a number of different unexpected exceptions.
 #)  Error ``DPY-1006: cursor is not open`` is now raised consistently when
     attempting to bind a closed cursor. Previously, thin mode would result in a
     segfault and thick mode would result in unusual errors.

src/oracledb/errors.py

@@ -286,6 +286,7 @@ def _raise_not_supported(feature: str) -> None:
 ERR_VECTOR_FORMAT_NOT_SUPPORTED = 3024
 ERR_OPERATION_NOT_SUPPORTED_ON_BFILE = 3025
 ERR_OPERATION_ONLY_SUPPORTED_ON_BFILE = 3026
+ERR_CURSOR_DIFF_CONNECTION = 3027
 
 # error numbers that result in DatabaseError
 ERR_TNS_ENTRY_NOT_FOUND = 4000
@@ -475,6 +476,9 @@ def _raise_not_supported(feature: str) -> None:
         "cannot connect to database (CONNECTION_ID={connection_id})."
     ),
     ERR_CONTENT_INVALID_AFTER_NUMBER: "invalid number (content after number)",
+    ERR_CURSOR_DIFF_CONNECTION: (
+        "binding a cursor from a different connection is not supported"
+    ),
     ERR_CURSOR_NOT_OPEN: "cursor is not open",
     ERR_CURSOR_HAS_BEEN_CLOSED: "cursor has been closed by the database",
     ERR_DBOBJECT_ATTR_MAX_SIZE_VIOLATED: (

src/oracledb/impl/base/connection.pyx

@@ -134,6 +134,8 @@ cdef class BaseConnImpl:
         elif db_type_num == DB_TYPE_NUM_CURSOR:
             if isinstance(value, (PY_TYPE_CURSOR, PY_TYPE_ASYNC_CURSOR)):
                 value._verify_open()
+                if value.connection._impl is not self:
+                    errors._raise_err(errors.ERR_CURSOR_DIFF_CONNECTION)
                 return value
         elif db_type_num == DB_TYPE_NUM_BOOLEAN:
             return bool(value)

tests/test_1300_cursor_var.py

@@ -427,8 +427,8 @@ def test_1317(self):
         ref_cursor = self.conn.cursor()
         ref_cursor.close()
         with self.assertRaisesFullCode("DPY-1006"):
-            self.cursor.callproc(
-                "pkg_testRefCursors.TestInCursor", [ref_cursor]
+            self.cursor.callfunc(
+                "pkg_testRefCursors.TestInCursor", str, [ref_cursor]
             )
 
     def test_1318(self):
@@ -442,6 +442,23 @@ def test_1318(self):
             ref_cursor = var.getvalue()
             ref_cursor.fetchall()
 
+    def test_1319(self):
+        "1319 - test binding cursor that is not from the same connection"
+        sql = """
+            declare
+                t_Cursor sys_refcursor;
+            begin
+                open t_Cursor for
+                    select 1319
+                    from dual;
+                :cursor := t_Cursor;
+            end;
+        """
+        conn = test_env.get_connection()
+        ref_cursor = conn.cursor()
+        with self.assertRaisesFullCode("DPY-3027"):
+            self.cursor.execute(sql, [ref_cursor])
+
 
 if __name__ == "__main__":
     test_env.run_test_cases()