Address comments

Author: SakiTakamachi

ext/oci8/oci8.c

@@ -574,7 +574,7 @@ void php_oci_column_hash_dtor(zval *data)
 	}
 
 	if (column->descid && !GC_DELREF(column->descid)) {
-		zend_list_close(column->descid);
+		zend_list_free(column->descid);
 	}
 
 	if (column->data) {

ext/oci8/tests/gh18873.phpt

@@ -0,0 +1,37 @@
+--TEST--
+GH-18873 (OCI_RETURN_LOBS flag causes oci8 to leak memory)
+--EXTENSIONS--
+oci8
+--SKIPIF--
+<?php
+require_once 'skipifconnectfailure.inc';
+?>
+--FILE--
+<?php
+require __DIR__.'/connect.inc';
+
+$expectedStr = str_repeat('a', 1_001);
+$sql = 'select concat(TO_CLOB(\'' . str_repeat('a', 1_000) . '\'), TO_CLOB(\'a\')) AS "v" from "DUAL"';
+
+$memUsages = array_flip(range(0, 100 - 1));
+foreach (array_keys($memUsages) as $k) {
+    $stid = oci_parse($c, $sql);
+    oci_execute($stid);
+    $row = oci_fetch_array($stid, \OCI_ASSOC | \OCI_RETURN_LOBS);
+    $res = $row['v'];
+
+    $memUsages[$k] = memory_get_usage();
+}
+
+$memUsages = array_slice($memUsages, 1, null, true);
+$memUsages = array_unique($memUsages);
+
+if (count($memUsages) !== 1) {
+    var_dump($memUsages);
+    throw new \Exception('memory leak detected');
+}
+
+echo "Done!\n";
+?>
+--EXPECT--
+Done!