Fix GH-11878: SQLite3 callback functions cause a memory leak with a callable array

In this test file, the free_obj handler is called with a refcount of 2,
caused by the fact it participates in a cycle and we do a GC_ADDREF() to
increase its refcount. We never decrease its refcount so it never
becomes 0 causing a memory leak.
This happens in shutdown. Whether it is actually useful to check the
refcount afterwards is up for debate.

Author: nielsdos

Zend/zend_objects_API.c

@@ -107,6 +107,9 @@ ZEND_API void ZEND_FASTCALL zend_objects_store_free_object_storage(zend_objects_
 					if (obj->handlers->free_obj != zend_object_std_dtor) {
 						GC_ADDREF(obj);
 						obj->handlers->free_obj(obj);
+						if (!GC_DELREF(obj)) {
+							zend_objects_store_del(obj);
+						}
 					}
 				}
 			}
@@ -120,6 +123,9 @@ ZEND_API void ZEND_FASTCALL zend_objects_store_free_object_storage(zend_objects_
 					GC_ADD_FLAGS(obj, IS_OBJ_FREE_CALLED);
 					GC_ADDREF(obj);
 					obj->handlers->free_obj(obj);
+					if (!GC_DELREF(obj)) {
+						zend_objects_store_del(obj);
+					}
 				}
 			}
 		} while (obj_ptr != end);

ext/sqlite3/tests/gh11878.phpt

@@ -0,0 +1,21 @@
+--TEST--
+GH-11878 (SQLite3 callback functions cause a memory leak with a callable array)
+--EXTENSIONS--
+sqlite3
+--FILE--
+<?php
+class Foo {
+    public $sqlite;
+    public function __construct() {
+        $this->sqlite = new SQLite3(":memory:");
+        $this->sqlite->createAggregate("indexes", array($this, "SQLiteIndex"), array($this, "SQLiteFinal"), 9);
+    }
+    public function SQLiteIndex() {}
+    public function SQLiteFinal() {}
+}
+
+$x = new Foo;
+?>
+Done
+--EXPECT--
+Done