Address review comments

Author: kocsismate

ext/ldap/ldap.c

@@ -87,7 +87,6 @@ typedef struct {
 typedef struct {
 	LDAPMessage *data;
 	BerElement  *ber;
-	zval         res;
 	zend_object std;
 } ldap_result_entry;
 
@@ -233,7 +232,7 @@ static void ldap_result_entry_free_obj(zend_object *obj)
 #define VERIFY_LDAP_RESULT_OPEN(lr) \
 { \
 	if (!lr->result) { \
-		zend_throw_error(NULL, "LDAP result has already been freed"); \
+		zend_throw_error(NULL, "LDAP result has already been closed"); \
 		RETURN_THROWS(); \
 	} \
 }
@@ -1167,7 +1166,6 @@ PHP_FUNCTION(ldap_connect)
 			size_t urllen = hostlen + sizeof( "ldap://:65535" );
 
 			if (port <= 0 || port > 65535) {
-				zval_ptr_dtor(return_value);
 				zend_argument_value_error(2, "must be between 1 and 65535");
 				RETURN_THROWS();
 			}

ext/ldap/tests/ldap_search_error.phpt

@@ -39,6 +39,18 @@ try {
     echo $exception->getMessage() . "\n";
 }
 
+try {
+    ldap_search($link, [], []);
+} catch (TypeError $exception) {
+    echo $exception->getMessage() . "\n";
+}
+
+try {
+    ldap_search($link, "", []);
+} catch (TypeError $exception) {
+    echo $exception->getMessage() . "\n";
+}
+
 ?>
 --EXPECTF--
 Warning: ldap_search(): Search: No such object in %s on line %d
@@ -49,3 +61,5 @@ bool(false)
 ldap_search(): Argument #1 ($ldap) cannot be empty
 ldap_search(): Argument #2 ($base) must have the same number of elements as the links array
 ldap_search(): Argument #3 ($filter) must have the same number of elements as the links array
+ldap_search(): Argument #2 ($base) must be of type string when argument #1 ($ldap) is an LDAP instance
+ldap_search(): Argument #3 ($filter) must be of type string when argument #1 ($ldap) is an LDAP instance