Free altname if not using it

StephenWall · StephenWall · commit 1a7f2d6863a5 · 2026-06-23T16:16:03.000-04:00
diff --git a/ext/openssl/openssl.c b/ext/openssl/openssl.c
@@ -1563,6 +1563,8 @@ PHP_FUNCTION(openssl_x509_parse)
 	ulong 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
`@@ -1563,6 +1563,8 @@ PHP_FUNCTION(openssl_x509_parse)`
`1563`	`1563`	`ulong altcount = zend_hash_num_elements(Z_ARRVAL_P(&altname));`
`1564`	`1564`	`if (altcount > 0) {`
`1565`	`1565`	`add_assoc_zval(return_value, "subjectAlternativeName", &altname);`
	`1566`	`+ } else {`
	`1567`	`+ zval_ptr_dtor(&altname);`
`1566`	`1568`	`}`
`1567`	`1569`	`if (cert_str) {`
`1568`	`1570`	`X509_free(cert);`