ext/ldap: fix leak on port overflow check.

devnexen · devnexen · commit 890c2f027b08 · 2025-05-24T22:44:21.000+01:00
diff --git a/ext/ldap/ldap.c b/ext/ldap/ldap.c
@@ -984,8 +984,6 @@ PHP_FUNCTION(ldap_connect)
 		RETURN_FALSE;
 	}
 
-	object_init_ex(return_value, ldap_link_ce);
-	ld = Z_LDAP_LINK_P(return_value);
 
 	{
 		int rc = LDAP_SUCCESS;
@@ -1015,6 +1013,8 @@ PHP_FUNCTION(ldap_connect)
 			LDAPG(tls_newctx) = false;
 		}
 #endif
+		object_init_ex(return_value, ldap_link_ce);
+		ld = Z_LDAP_LINK_P(return_value);
 
 #ifdef LDAP_API_FEATURE_X_OPENLDAP
 		/* ldap_init() is deprecated, use ldap_initialize() instead.

Original file line number	Diff line number	Diff line change
`@@ -984,8 +984,6 @@ PHP_FUNCTION(ldap_connect)`
`984`	`984`	`RETURN_FALSE;`
`985`	`985`	`}`
`986`	`986`
`987`		`- object_init_ex(return_value, ldap_link_ce);`
`988`		`- ld = Z_LDAP_LINK_P(return_value);`
`989`	`987`
`990`	`988`	`{`
`991`	`989`	`int rc = LDAP_SUCCESS;`
`@@ -1015,6 +1013,8 @@ PHP_FUNCTION(ldap_connect)`
`1015`	`1013`	`LDAPG(tls_newctx) = false;`
`1016`	`1014`	`}`
`1017`	`1015`	`#endif`
	`1016`	`+ object_init_ex(return_value, ldap_link_ce);`
	`1017`	`+ ld = Z_LDAP_LINK_P(return_value);`
`1018`	`1018`
`1019`	`1019`	`#ifdef LDAP_API_FEATURE_X_OPENLDAP`
`1020`	`1020`	`/* ldap_init() is deprecated, use ldap_initialize() instead.`