Merge branch 'PHP-8.5'

TimWolla · TimWolla · commit 7415c504723c · 2026-04-12T18:44:11.000+02:00
* PHP-8.5: Fix GH-21730: Mt19937::__debugInfo() leaks state HashTable when the serialize callback fails (#21733)
diff --git a/ext/random/engine_mt19937.c b/ext/random/engine_mt19937.c
@@ -389,11 +389,11 @@ PHP_METHOD(Random_Engine_Mt19937, __debugInfo)
 
 	if (engine->engine.algo->serialize) {
 		array_init(&t);
+		zend_hash_str_add(Z_ARR_P(return_value), "__states", strlen("__states"), &t);
 		if (!engine->engine.algo->serialize(engine->engine.state, Z_ARRVAL(t))) {
 			zend_throw_exception(NULL, "Engine serialize failed", 0);
 			RETURN_THROWS();
 		}
-		zend_hash_str_add(Z_ARR_P(return_value), "__states", strlen("__states"), &t);
 	}
 }
 /* }}} */

Original file line number	Diff line number	Diff line change
`@@ -389,11 +389,11 @@ PHP_METHOD(Random_Engine_Mt19937, __debugInfo)`
`389`	`389`
`390`	`390`	`if (engine->engine.algo->serialize) {`
`391`	`391`	`array_init(&t);`
	`392`	`+ zend_hash_str_add(Z_ARR_P(return_value), "__states", strlen("__states"), &t);`
`392`	`393`	`if (!engine->engine.algo->serialize(engine->engine.state, Z_ARRVAL(t))) {`
`393`	`394`	`zend_throw_exception(NULL, "Engine serialize failed", 0);`
`394`	`395`	`RETURN_THROWS();`
`395`	`396`	`}`
`396`		`- zend_hash_str_add(Z_ARR_P(return_value), "__states", strlen("__states"), &t);`
`397`	`397`	`}`
`398`	`398`	`}`
`399`	`399`	`/* }}} */`