From: Patrick Allaert <patrickallaert@php.net>
Date: Sun, 7 Jun 2009 20:00:32 +0000 (+0000)
Subject: Fixed ldap_(first|next)_attribute() function producing 2 warning messages in case... 
X-Git-Tag: php-5.4.0alpha1~191^2~3391
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=77693a80b7ed0975b1c2455cca4f2f091e526071;p=php

Fixed ldap_(first|next)_attribute() function producing 2 warning messages in case wrong parameters given
---

diff --git a/ext/ldap/ldap.c b/ext/ldap/ldap.c
index db10408176..e1c4cf885d 100644
--- a/ext/ldap/ldap.c
+++ b/ext/ldap/ldap.c
@@ -1051,8 +1051,8 @@ PHP_FUNCTION(ldap_first_attribute)
 	ldap_resultentry *resultentry;
 	char *attribute;
 
-	if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "rr", &link, &result_entry) == FAILURE) {
-		WRONG_PARAM_COUNT;
+	if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "rr", &link, &result_entry) != SUCCESS) {
+		return;
 	}
 
 	ZEND_FETCH_RESOURCE(ld, ldap_linkdata *, &link, -1, "ldap link", le_link);
@@ -1078,8 +1078,8 @@ PHP_FUNCTION(ldap_next_attribute)
 	ldap_resultentry *resultentry;
 	char *attribute;
 
-	if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "rr", &link, &result_entry) == FAILURE) {
-		WRONG_PARAM_COUNT;
+	if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "rr", &link, &result_entry) != SUCCESS) {
+		return;
 	}
 
 	ZEND_FETCH_RESOURCE(ld, ldap_linkdata *, &link, -1, "ldap link", le_link);