ext/ldap: Use size_t type instead of int type

Girgias · Girgias · commit 558327390649 · 2025-08-06T16:36:13.000+01:00
diff --git a/ext/ldap/ldap.c b/ext/ldap/ldap.c
@@ -719,15 +719,13 @@ static int php_ldap_control_from_array(LDAP *ld, LDAPControl** ctrl, const HashT
 		zend_string_release(tmpstring);
 	}
 	if (tmpstrings1 != NULL) {
-		int i;
-		for (i = 0; i < num_tmpstrings1; ++i) {
+		for (size_t i = 0; i < num_tmpstrings1; ++i) {
 			zend_string_release(tmpstrings1[i]);
 		}
 		efree(tmpstrings1);
 	}
 	if (tmpstrings2 != NULL) {
-		int i;
-		for (i = 0; i < num_tmpstrings2; ++i) {
+		for (size_t i = 0; i < num_tmpstrings2; ++i) {
 			zend_string_release(tmpstrings2[i]);
 		}
 		efree(tmpstrings2);

Original file line number	Diff line number	Diff line change
`@@ -719,15 +719,13 @@ static int php_ldap_control_from_array(LDAP ld, LDAPControl* ctrl, const HashT`
`719`	`719`	`zend_string_release(tmpstring);`
`720`	`720`	`}`
`721`	`721`	`if (tmpstrings1 != NULL) {`
`722`		`- int i;`
`723`		`- for (i = 0; i < num_tmpstrings1; ++i) {`
	`722`	`+ for (size_t i = 0; i < num_tmpstrings1; ++i) {`
`724`	`723`	`zend_string_release(tmpstrings1[i]);`
`725`	`724`	`}`
`726`	`725`	`efree(tmpstrings1);`
`727`	`726`	`}`
`728`	`727`	`if (tmpstrings2 != NULL) {`
`729`		`- int i;`
`730`		`- for (i = 0; i < num_tmpstrings2; ++i) {`
	`728`	`+ for (size_t i = 0; i < num_tmpstrings2; ++i) {`
`731`	`729`	`zend_string_release(tmpstrings2[i]);`
`732`	`730`	`}`
`733`	`731`	`efree(tmpstrings2);`