Implement ldap_exop_sync

Author: kocsismate

ext/ldap/ldap.c

@@ -3883,9 +3883,7 @@ PHP_FUNCTION(ldap_8859_to_t61)
 
 /* {{{ Extended operations, Pierangelo Masarati */
 #ifdef HAVE_LDAP_EXTENDED_OPERATION_S
-/* {{{ Extended operation */
-PHP_FUNCTION(ldap_exop)
-{
+static void php_ldap_exop(INTERNAL_FUNCTION_PARAMETERS, bool force_sync) {
 	zval *serverctrls = NULL;
 	zval *link, *retdata = NULL, *retoid = NULL;
 	char *lretoid = NULL;
@@ -3919,7 +3917,7 @@ PHP_FUNCTION(ldap_exop)
 		}
 	}
 
-	if (retdata) {
+	if (force_sync || retdata) {
 		/* synchronous call */
 		rc = ldap_extended_operation_s(ld->link, ZSTR_VAL(reqoid),
 			lreqdata.bv_len > 0 ? &lreqdata: NULL,
@@ -3978,12 +3976,23 @@ PHP_FUNCTION(ldap_exop)
 	result = Z_LDAP_RESULT_P(return_value);
 	result->result = ldap_res;
 
-	cleanup:
+cleanup:
 	if (lserverctrls) {
 		_php_ldap_controls_free(&lserverctrls);
 	}
 }
+
+/* {{{ Extended operation */
+PHP_FUNCTION(ldap_exop)
+{
+	php_ldap_exop(INTERNAL_FUNCTION_PARAM_PASSTHRU, false);
+}
 /* }}} */
+
+PHP_FUNCTION(ldap_exop_sync)
+{
+	php_ldap_exop(INTERNAL_FUNCTION_PARAM_PASSTHRU, true);
+}
 #endif
 
 #ifdef HAVE_LDAP_PASSWD

ext/ldap/ldap.stub.php

@@ -777,6 +777,11 @@ function ldap_8859_to_t61(string $value): string|false {}
      * @param string $response_oid
      */
     function ldap_exop(LDAP\Connection $ldap, string $request_oid, ?string $request_data = null, ?array $controls = NULL, &$response_data = UNKNOWN, &$response_oid = null): LDAP\Result|bool {}
+    /**
+     * @param string $response_data
+     * @param string $response_oid
+     */
+    function ldap_exop_sync(LDAP\Connection $ldap, string $request_oid, ?string $request_data = null, ?array $controls = NULL, &$response_data = null, &$response_oid = null): LDAP\Result|bool {}
     #endif
 
     #ifdef HAVE_LDAP_PASSWD

ext/ldap/ldap_arginfo.h

@@ -41,8 +41,13 @@ var_dump(
     ldap_exop($link, LDAP_EXOP_WHO_AM_I, NULL, NULL, $retdata, $retoid),
     $retdata,
     $retoid,
+    ldap_exop_sync($link, LDAP_EXOP_WHO_AM_I, NULL, NULL, $retdata, $retoid),
+    $retdata,
+    $retoid,
     ldap_exop($link, LDAP_EXOP_WHO_AM_I, NULL, [['oid' => LDAP_CONTROL_PROXY_AUTHZ, 'value' => "dn:cn=userA,$base"]], $retdata),
     $retdata,
+    ldap_exop_sync($link, LDAP_EXOP_WHO_AM_I, NULL, [['oid' => LDAP_CONTROL_PROXY_AUTHZ, 'value' => "dn:cn=userA,$base"]], $retdata),
+    $retdata,
     $r = ldap_exop($link, LDAP_EXOP_WHO_AM_I),
     ldap_parse_exop($link, $r, $retdata2),
     $retdata2,
@@ -67,6 +72,11 @@ bool(true)
 string(%d) "dn:%s"
 string(0) ""
 bool(true)
+string(%d) "dn:%s"
+string(0) ""
+bool(true)
+string(%d) "dn:cn=user%s"
+bool(true)
 string(%d) "dn:cn=user%s"
 object(LDAP\Result)#%d (0) {
 }