Eliminated small memory leak with production of all data frames.

anthony-tuininga · anthony-tuininga · commit b65e0098a115 · 2025-03-19T10:38:44.000-06:00
diff --git a/doc/src/release_notes.rst b/doc/src/release_notes.rst
@@ -57,6 +57,7 @@ Common Changes
     - avoid memory allocation/free cycles for decimal data
     - eliminated memory leak if OracleDataFrame is not converted to an external
       data frame
+    - eliminated small memory leak with production of all data frames
 
 #)  Fixed bug when NUMBER data is fetched with
     :meth:`Connection.fetch_df_all()` or :meth:`Connection.fetch_df_batches()`
diff --git a/src/oracledb/interchange/nanoarrow_bridge.pyx b/src/oracledb/interchange/nanoarrow_bridge.pyx
@@ -336,11 +336,10 @@ cdef class OracleArrowArray:
         cdef:
             int64_t n_buffers = self.arrow_array.n_buffers
             ArrowBufferView *buffer
-            ArrowArrayView *view
-        view = <ArrowArrayView*> cpython.PyMem_Malloc(sizeof(ArrowArrayView))
-        _check_nanoarrow(ArrowArrayViewInitFromSchema(view, self.arrow_schema,
+            ArrowArrayView view
+        _check_nanoarrow(ArrowArrayViewInitFromSchema(&view, self.arrow_schema,
                                                       NULL))
-        _check_nanoarrow(ArrowArrayViewSetArray(view, self.arrow_array, NULL))
+        _check_nanoarrow(ArrowArrayViewSetArray(&view, self.arrow_array, NULL))
 
         # initialize all buffers to None to begin with
         buffers = {
