Free altname if not using it

StephenWall · StephenWall · commit b2f67ff6d109 · 2026-01-14T13:29:25.000-05:00
diff --git a/ext/openssl/openssl.c b/ext/openssl/openssl.c
@@ -1155,6 +1155,8 @@ PHP_FUNCTION(openssl_x509_parse)
 	zend_long altcount = zend_hash_num_elements(Z_ARRVAL_P(&altname));
 	if (altcount > 0) {
 		add_assoc_zval(return_value, "subjectAlternativeName", &altname);
+	} else {
+		zval_ptr_dtor(&altname);
 	}
 	if (cert_str) {
 		X509_free(cert);

Original file line number	Diff line number	Diff line change
`@@ -1155,6 +1155,8 @@ PHP_FUNCTION(openssl_x509_parse)`
`1155`	`1155`	`zend_long altcount = zend_hash_num_elements(Z_ARRVAL_P(&altname));`
`1156`	`1156`	`if (altcount > 0) {`
`1157`	`1157`	`add_assoc_zval(return_value, "subjectAlternativeName", &altname);`
	`1158`	`+ } else {`
	`1159`	`+ zval_ptr_dtor(&altname);`
`1158`	`1160`	`}`
`1159`	`1161`	`if (cert_str) {`
`1160`	`1162`	`X509_free(cert);`