Merge branch 'PHP-8.3' into PHP-8.4

ndossche · ndossche · commit 3c1c7a0fe422 · 2025-11-04T08:33:31.000+01:00
* PHP-8.3: random: Fix memory leak when serialization fails (#20383)
diff --git a/ext/random/engine_mt19937.c b/ext/random/engine_mt19937.c
@@ -330,11 +330,11 @@ PHP_METHOD(Random_Engine_Mt19937, __serialize)
 
 	/* state */
 	array_init(&t);
+	zend_hash_next_index_insert(Z_ARRVAL_P(return_value), &t);
 	if (!engine->engine.algo->serialize(engine->engine.state, Z_ARRVAL(t))) {
 		zend_throw_exception(NULL, "Engine serialize failed", 0);
 		RETURN_THROWS();
 	}
-	zend_hash_next_index_insert(Z_ARRVAL_P(return_value), &t);
 }
 /* }}} */
 

Original file line number	Diff line number	Diff line change
`@@ -330,11 +330,11 @@ PHP_METHOD(Random_Engine_Mt19937, __serialize)`
`330`	`330`
`331`	`331`	`/* state */`
`332`	`332`	`array_init(&t);`
	`333`	`+ zend_hash_next_index_insert(Z_ARRVAL_P(return_value), &t);`
`333`	`334`	`if (!engine->engine.algo->serialize(engine->engine.state, Z_ARRVAL(t))) {`
`334`	`335`	`zend_throw_exception(NULL, "Engine serialize failed", 0);`
`335`	`336`	`RETURN_THROWS();`
`336`	`337`	`}`
`337`		`- zend_hash_next_index_insert(Z_ARRVAL_P(return_value), &t);`
`338`	`338`	`}`
`339`	`339`	`/* }}} */`
`340`	`340`