]> granicus.if.org Git - php/commitdiff
Fixed ldap_(first|next)_attribute() function producing 2 warning messages in case...
authorPatrick Allaert <patrickallaert@php.net>
Sun, 7 Jun 2009 20:00:32 +0000 (20:00 +0000)
committerPatrick Allaert <patrickallaert@php.net>
Sun, 7 Jun 2009 20:00:32 +0000 (20:00 +0000)
ext/ldap/ldap.c

index db10408176cc29831849bc2a0358e1018ff381e2..e1c4cf885ddf24730d4dc32acc154663186d5a00 100644 (file)
@@ -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);