Skip to content

Commit

Permalink
Mark ldap_exop() param as UNKNOWN
Browse files Browse the repository at this point in the history
This function actually has very different behavior depending on
whether $response_data is passed or not.
  • Loading branch information
nikic committed Oct 14, 2020
1 parent 0832376 commit 1610e81
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion ext/ldap/ldap.stub.php
Original file line number Diff line number Diff line change
Expand Up @@ -279,7 +279,7 @@ function ldap_8859_to_t61(string $value): string|false {}
* @param string $response_oid
* @return resource|bool
*/
function ldap_exop($ldap, string $request_oid, ?string $request_data = null, ?array $controls = [], &$response_data = null, &$response_oid = null) {}
function ldap_exop($ldap, string $request_oid, ?string $request_data = null, ?array $controls = [], &$response_data = UNKNOWN, &$response_oid = null) {}
#endif

#ifdef HAVE_LDAP_PASSWD
Expand Down
4 changes: 2 additions & 2 deletions ext/ldap/ldap_arginfo.h
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/* This is a generated file, edit the .stub.php file instead.
* Stub hash: 42f7118d8380424cd4178759cae6cb47b6d9b44f */
* Stub hash: 0c02f5e91a47e9664c8c41332bf8040e93592753 */

#if defined(HAVE_ORALDAP)
ZEND_BEGIN_ARG_INFO_EX(arginfo_ldap_connect, 0, 0, 0)
Expand Down Expand Up @@ -297,7 +297,7 @@ ZEND_BEGIN_ARG_INFO_EX(arginfo_ldap_exop, 0, 0, 2)
ZEND_ARG_TYPE_INFO(0, request_oid, IS_STRING, 0)
ZEND_ARG_TYPE_INFO_WITH_DEFAULT_VALUE(0, request_data, IS_STRING, 1, "null")
ZEND_ARG_TYPE_INFO_WITH_DEFAULT_VALUE(0, controls, IS_ARRAY, 1, "[]")
ZEND_ARG_INFO_WITH_DEFAULT_VALUE(1, response_data, "null")
ZEND_ARG_INFO(1, response_data)
ZEND_ARG_INFO_WITH_DEFAULT_VALUE(1, response_oid, "null")
ZEND_END_ARG_INFO()
#endif
Expand Down

0 comments on commit 1610e81

Please sign in to comment.