Revert r1140069:

    Fix load order dependencies in LDAP code by switching to use of APR
    optional functions for the inter-module API...

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/branches/revert-ap-ldap@1150164 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/include/ap_ldap.h.in b/include/ap_ldap.h.in
index 409d4c67a55ef48e6040628686f2d64314207b1d..130c6dd016cc61096cfb926e66f2824b8b50408c 100644 (file)
--- a/include/ap_ldap.h.in
+++ b/include/ap_ldap.h.in
@@ -30,7 +30,26 @@
  * @{
  */
 
-#include "apr_optional.h"
+/* Create a set of LDAP_DECLARE macros with appropriate export 
+ * and import tags for the platform
+ */
+#if !defined(WIN32)
+#define LDAP_DECLARE(type)            type
+#define LDAP_DECLARE_NONSTD(type)     type
+#define LDAP_DECLARE_DATA
+#elif defined(LDAP_DECLARE_STATIC)
+#define LDAP_DECLARE(type)            type __stdcall
+#define LDAP_DECLARE_NONSTD(type)     type
+#define LDAP_DECLARE_DATA
+#elif defined(LDAP_DECLARE_EXPORT)
+#define LDAP_DECLARE(type)            __declspec(dllexport) type __stdcall
+#define LDAP_DECLARE_NONSTD(type)     __declspec(dllexport) type
+#define LDAP_DECLARE_DATA             __declspec(dllexport)
+#else
+#define LDAP_DECLARE(type)            __declspec(dllimport) type __stdcall
+#define LDAP_DECLARE_NONSTD(type)     __declspec(dllimport) type
+#define LDAP_DECLARE_DATA             __declspec(dllimport)
+#endif
 
 /* identify the LDAP toolkit used */
 #define AP_HAS_NETSCAPE_LDAPSDK  @ap_has_ldap_netscape@

diff --git a/include/ap_ldap.hw b/include/ap_ldap.hw
index b64b4f4ae436345d271c606c36e069047c67cf4f..36e21abd860bf836f9ee89d311f564fea9758c41 100644 (file)
--- a/include/ap_ldap.hw
+++ b/include/ap_ldap.hw
@@ -30,7 +30,26 @@
  * @{
  */
 
-#include "apr_optional.h"
+/* Create a set of LDAP_DECLARE macros with appropriate export 
+ * and import tags for the platform
+ */
+#if !defined(WIN32)
+#define LDAP_DECLARE(type)            type
+#define LDAP_DECLARE_NONSTD(type)     type
+#define LDAP_DECLARE_DATA
+#elif defined(LDAP_DECLARE_STATIC)
+#define LDAP_DECLARE(type)            type __stdcall
+#define LDAP_DECLARE_NONSTD(type)     type
+#define LDAP_DECLARE_DATA
+#elif defined(LDAP_DECLARE_EXPORT)
+#define LDAP_DECLARE(type)            __declspec(dllexport) type __stdcall
+#define LDAP_DECLARE_NONSTD(type)     __declspec(dllexport) type
+#define LDAP_DECLARE_DATA             __declspec(dllexport)
+#else
+#define LDAP_DECLARE(type)            __declspec(dllimport) type __stdcall
+#define LDAP_DECLARE_NONSTD(type)     __declspec(dllimport) type
+#define LDAP_DECLARE_DATA             __declspec(dllimport)
+#endif
 
 /* this will be defined if LDAP support was compiled into apr-util */
 #define AP_HAS_LDAP               1

diff --git a/include/ap_ldap_init.h b/include/ap_ldap_init.h
index 440f31835053b5bb0e2a7e0dd2d0d30e9b88a929..0fe0bcc6c1c3593649da924a5140c4baaac2411b 100644 (file)
--- a/include/ap_ldap_init.h
+++ b/include/ap_ldap_init.h
@@ -86,12 +86,10 @@ extern "C" {
  * ap_ldap_set_option() AP_LDAP_OPT_TLS_CERT option for details.
  * @param result_err The returned result
  */
-APR_DECLARE_OPTIONAL_FN(int, ap_ldap_ssl_init, 
-                        (apr_pool_t *pool,
-                         const char *cert_auth_file,
-                         int cert_file_type,
-                         ap_ldap_err_t **result_err));
-
+LDAP_DECLARE(int) ap_ldap_ssl_init(apr_pool_t *pool,
+                                      const char *cert_auth_file,
+                                      int cert_file_type,
+                                      ap_ldap_err_t **result_err);
 
 /**
  * LDAP SSL De-Initialise function
@@ -102,7 +100,7 @@ APR_DECLARE_OPTIONAL_FN(int, ap_ldap_ssl_init,
  * @todo currently we do not check whether ap_ldap_ssl_init()
  * has been called first - we probably should.
  */
-APR_DECLARE_OPTIONAL_FN(int, ap_ldap_ssl_deinit, (void));
+LDAP_DECLARE(int) ap_ldap_ssl_deinit(void);
 
 /**
  * LDAP initialise function
@@ -138,12 +136,12 @@ APR_DECLARE_OPTIONAL_FN(int, ap_ldap_ssl_deinit, (void));
  * @param secure The security mode to set
  * @param result_err The returned result
  */
-APR_DECLARE_OPTIONAL_FN(int, ap_ldap_init, (apr_pool_t *pool, 
-                                            LDAP **ldap,
-                                            const char *hostname,
-                                            int portno,
-                                            int secure,
-                                            ap_ldap_err_t **result_err));
+LDAP_DECLARE(int) ap_ldap_init(apr_pool_t *pool,
+                                  LDAP **ldap,
+                                  const char *hostname,
+                                  int portno,
+                                  int secure,
+                                  ap_ldap_err_t **result_err);
 
 /**
  * LDAP info function
@@ -153,8 +151,8 @@ APR_DECLARE_OPTIONAL_FN(int, ap_ldap_init, (apr_pool_t *pool,
  * @param pool The pool to use
  * @param result_err The returned result
  */
-APR_DECLARE_OPTIONAL_FN(int, ap_ldap_info, (apr_pool_t *pool,
-                                            ap_ldap_err_t **result_err));
+LDAP_DECLARE(int) ap_ldap_info(apr_pool_t *pool,
+                                  ap_ldap_err_t **result_err);
 
 #ifdef __cplusplus
 }

diff --git a/include/ap_ldap_option.h b/include/ap_ldap_option.h
index b0c30af23e6fa7b1c91187dec9f81a248a0f8815..6e73e5044fe5fc59b30f63695a77ed2683d8dbd6 100644 (file)
--- a/include/ap_ldap_option.h
+++ b/include/ap_ldap_option.h
@@ -214,11 +214,11 @@ struct ap_ldap_opt_tls_cert_t {
  * @param result_err The ap_ldap_err_t structure contained detailed results
  *        of the operation.
  */
-APR_DECLARE_OPTIONAL_FN(int, ap_ldap_get_option, (apr_pool_t *pool,
-                                                  LDAP *ldap,
-                                                  int option,
-                                                  void *outvalue,
-                                                  ap_ldap_err_t **result_err));
+LDAP_DECLARE(int) ap_ldap_get_option(apr_pool_t *pool,
+                                        LDAP *ldap,
+                                        int option,
+                                        void *outvalue,
+                                        ap_ldap_err_t **result_err);
 
 /**
  * LDAP set option function
@@ -236,11 +236,11 @@ APR_DECLARE_OPTIONAL_FN(int, ap_ldap_get_option, (apr_pool_t *pool,
  * @param result_err The ap_ldap_err_t structure contained detailed results
  *        of the operation.
  */
-APR_DECLARE_OPTIONAL_FN(int, ap_ldap_set_option, (apr_pool_t *pool,
-                                                  LDAP *ldap,
-                                                  int option,
-                                                  const void *invalue,
-                                                  ap_ldap_err_t **result_err));
+LDAP_DECLARE(int) ap_ldap_set_option(apr_pool_t *pool,
+                                        LDAP *ldap,
+                                        int option,
+                                        const void *invalue,
+                                        ap_ldap_err_t **result_err);
 
 #ifdef __cplusplus
 }

diff --git a/include/ap_ldap_rebind.h b/include/ap_ldap_rebind.h
index f0090d30e95564f4acd823b2619f56c20c098957..7bc5a5cda963c43880238e775fb2fbde761c59b4 100644 (file)
--- a/include/ap_ldap_rebind.h
+++ b/include/ap_ldap_rebind.h
@@ -48,7 +48,7 @@
  * This function creates the lock for controlling access to the xref list..
  * @param pool Pool to use when creating the xref_lock.
  */
-APR_DECLARE_OPTIONAL_FN(apr_status_t, ap_ldap_rebind_init, (apr_pool_t *pool));
+LDAP_DECLARE(apr_status_t) ap_ldap_rebind_init(apr_pool_t *pool);
 
 
 /**
@@ -72,10 +72,10 @@ APR_DECLARE_OPTIONAL_FN(apr_status_t, ap_ldap_rebind_init, (apr_pool_t *pool));
  * @param bindPW The bind Password to be used for any binds while 
  *               chasing referrals on this ldap connection.
  */
-APR_DECLARE_OPTIONAL_FN(apr_status_t, ap_ldap_rebind_add, (apr_pool_t *pool,
-                                                           LDAP *ld,
-                                                           const char *bindDN,
-                                                           const char *bindPW));
+LDAP_DECLARE(apr_status_t) ap_ldap_rebind_add(apr_pool_t *pool,
+                                                 LDAP *ld,
+                                                 const char *bindDN,
+                                                 const char *bindPW);
 
 /**
  * LDAP rebind_remove function
@@ -88,7 +88,7 @@ APR_DECLARE_OPTIONAL_FN(apr_status_t, ap_ldap_rebind_add, (apr_pool_t *pool,
  *
  * @param ld The LDAP connectionhandle
  */
-APR_DECLARE_OPTIONAL_FN(apr_status_t, ap_ldap_rebind_remove, (LDAP *ld));
+LDAP_DECLARE(apr_status_t) ap_ldap_rebind_remove(LDAP *ld);
 
 #endif /* AP_HAS_LDAP */
 

diff --git a/include/ap_ldap_url.h b/include/ap_ldap_url.h
index 2c441dd8471b3ecdab1355c762461cccedcee89d..996b4b5a9aea622fa0160b1eeea619d3c4fdfac3 100644 (file)
--- a/include/ap_ldap_url.h
+++ b/include/ap_ldap_url.h
@@ -31,8 +31,6 @@
 #include "apu.h"
 #include "apr_pools.h"
 
-#include "apr_optional.h"
-
 #if defined(DOXYGEN)
 #include "ap_ldap.h"
 #endif
@@ -73,19 +71,19 @@ typedef struct ap_ldap_url_desc_t {
  * Is this URL an ldap url? ldap://
  * @param url The url to test
  */
-APR_DECLARE_OPTIONAL_FN(int, ap_ldap_is_ldap_url, (const char *url));
+LDAP_DECLARE(int) ap_ldap_is_ldap_url(const char *url);
 
 /**
  * Is this URL an SSL ldap url? ldaps://
  * @param url The url to test
  */
-APR_DECLARE_OPTIONAL_FN(int, ap_ldap_is_ldaps_url, (const char *url));
+LDAP_DECLARE(int) ap_ldap_is_ldaps_url(const char *url);
 
 /**
  * Is this URL an ldap socket url? ldapi://
  * @param url The url to test
  */
-APR_DECLARE_OPTIONAL_FN(int, ap_ldap_is_ldapi_url, (const char *url));
+LDAP_DECLARE(int) ap_ldap_is_ldapi_url(const char *url);
 
 /**
  * Parse an LDAP URL.
@@ -94,10 +92,10 @@ APR_DECLARE_OPTIONAL_FN(int, ap_ldap_is_ldapi_url, (const char *url));
  * @param ludpp The structure to return the exploded URL
  * @param result_err The result structure of the operation
  */
-APR_DECLARE_OPTIONAL_FN(int, ap_ldap_url_parse_ext, (apr_pool_t *pool,
-                                                     const char *url_in,
-                                                     ap_ldap_url_desc_t **ludpp,
-                                                     ap_ldap_err_t **result_err));
+LDAP_DECLARE(int) ap_ldap_url_parse_ext(apr_pool_t *pool,
+                                           const char *url_in,
+                                           ap_ldap_url_desc_t **ludpp,
+                                           ap_ldap_err_t **result_err);
 
 /**
  * Parse an LDAP URL.
@@ -106,10 +104,10 @@ APR_DECLARE_OPTIONAL_FN(int, ap_ldap_url_parse_ext, (apr_pool_t *pool,
  * @param ludpp The structure to return the exploded URL
  * @param result_err The result structure of the operation
  */
-APR_DECLARE_OPTIONAL_FN(int, ap_ldap_url_parse, (apr_pool_t *pool,
-                                                 const char *url_in,
-                                                 ap_ldap_url_desc_t **ludpp,
-                                                 ap_ldap_err_t **result_err));
+LDAP_DECLARE(int) ap_ldap_url_parse(apr_pool_t *pool,
+                                       const char *url_in,
+                                       ap_ldap_url_desc_t **ludpp,
+                                       ap_ldap_err_t **result_err);
 
 #ifdef __cplusplus
 }

diff --git a/modules/aaa/mod_authnz_ldap.c b/modules/aaa/mod_authnz_ldap.c
index 426f3f60ad2113973df25784d9316eb1fa2d0c5f..95bec065e43aa0de5fb10a11e2473d8738ef626f 100644 (file)
--- a/modules/aaa/mod_authnz_ldap.c
+++ b/modules/aaa/mod_authnz_ldap.c
@@ -111,7 +111,6 @@ static APR_OPTIONAL_FN_TYPE(uldap_cache_check_subgroups) *util_ldap_cache_check_
 static APR_OPTIONAL_FN_TYPE(uldap_cache_checkuserid) *util_ldap_cache_checkuserid;
 static APR_OPTIONAL_FN_TYPE(uldap_cache_getuserdn) *util_ldap_cache_getuserdn;
 static APR_OPTIONAL_FN_TYPE(uldap_ssl_supported) *util_ldap_ssl_supported;
-static APR_OPTIONAL_FN_TYPE(ap_ldap_url_parse) *util_ldap_url_parse;
 
 static apr_hash_t *charset_conversions = NULL;
 static char *to_charset = NULL;           /* UTF-8 identifier derived from the charset.conv file */
@@ -1339,7 +1338,7 @@ static const char *mod_auth_ldap_parse_url(cmd_parms *cmd,
 
     authn_ldap_config_t *sec = config;
 
-    rc = util_ldap_url_parse(cmd->pool, url, &(urld), &(result));
+    rc = ap_ldap_url_parse(cmd->pool, url, &(urld), &(result));
     if (rc != APR_SUCCESS) {
         return result->reason;
     }
@@ -1781,7 +1780,6 @@ static void ImportULDAPOptFn(void)
     util_ldap_cache_getuserdn   = APR_RETRIEVE_OPTIONAL_FN(uldap_cache_getuserdn);
     util_ldap_ssl_supported     = APR_RETRIEVE_OPTIONAL_FN(uldap_ssl_supported);
     util_ldap_cache_check_subgroups = APR_RETRIEVE_OPTIONAL_FN(uldap_cache_check_subgroups);
-    util_ldap_url_parse = APR_RETRIEVE_OPTIONAL_FN(ap_ldap_url_parse);
 }
 
 static void register_hooks(apr_pool_t *p)

diff --git a/modules/ldap/ldap_private.h b/modules/ldap/ldap_private.h
deleted file mode 100644 (file)
index ad6263b..0000000
--- a/modules/ldap/ldap_private.h
+++ /dev/null
@@ -1,74 +0,0 @@
-/* Licensed to the Apache Software Foundation (ASF) under one or more
- * contributor license agreements.  See the NOTICE file distributed with
- * this work for additional information regarding copyright ownership.
- * The ASF licenses this file to You under the Apache License, Version 2.0
- * (the "License"); you may not use this file except in compliance with
- * the License.  You may obtain a copy of the License at
- *
- *     http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-#ifndef AP_LDAP_PRIVATE_H
-#define AP_LDAP_PRIVATE_H
-
-#define LDAP_DECLARE(x) x
-
-#include "ap_ldap.h"
-
-/* Private declarations of API functions accessible only internally by
- * mod_ldap; these are all exported and the corresponding API docs are
- * in ap_ldap_*.h. */
-
-LDAP_DECLARE(int) ap_ldap_get_option(apr_pool_t *pool,
-                                     LDAP *ldap,
-                                     int option,
-                                     void *outvalue,
-                                     ap_ldap_err_t **result_err);
-
-LDAP_DECLARE(int) ap_ldap_set_option(apr_pool_t *pool,
-                                     LDAP *ldap,
-                                     int option,
-                                     const void *invalue,
-                                     ap_ldap_err_t **result_err);
-
-LDAP_DECLARE(int) ap_ldap_ssl_init(apr_pool_t *pool,
-                                      const char *cert_auth_file,
-                                      int cert_file_type,
-                                      ap_ldap_err_t **result_err);
-LDAP_DECLARE(int) ap_ldap_ssl_deinit(void);
-LDAP_DECLARE(int) ap_ldap_init(apr_pool_t *pool,
-                                  LDAP **ldap,
-                                  const char *hostname,
-                                  int portno,
-                                  int secure,
-                                  ap_ldap_err_t **result_err);
-
-LDAP_DECLARE(int) ap_ldap_info(apr_pool_t *pool,
-                                  ap_ldap_err_t **result_err);
-
-LDAP_DECLARE(int) ap_ldap_is_ldap_url(const char *url);
-LDAP_DECLARE(int) ap_ldap_is_ldaps_url(const char *url);
-LDAP_DECLARE(int) ap_ldap_is_ldapi_url(const char *url);
-LDAP_DECLARE(int) ap_ldap_url_parse_ext(apr_pool_t *pool,
-                                           const char *url_in,
-                                           ap_ldap_url_desc_t **ludpp,
-                                           ap_ldap_err_t **result_err);
-LDAP_DECLARE(int) ap_ldap_url_parse(apr_pool_t *pool,
-                                       const char *url_in,
-                                       ap_ldap_url_desc_t **ludpp,
-                                       ap_ldap_err_t **result_err);
-
-LDAP_DECLARE(apr_status_t) ap_ldap_rebind_init(apr_pool_t *pool);
-LDAP_DECLARE(apr_status_t) ap_ldap_rebind_add(apr_pool_t *pool,
-                                                 LDAP *ld,
-                                                 const char *bindDN,
-                                                 const char *bindPW);
-LDAP_DECLARE(apr_status_t) ap_ldap_rebind_remove(LDAP *ld);
-
-#endif /* AP_LDAP_PRIVATE_H */

diff --git a/modules/ldap/util_ldap.c b/modules/ldap/util_ldap.c
index 9427b69a2885c29b519bf38fa104eb613dc61252..4b19aaae58ebc843ceede8bcbdd54e2a9e5207a2 100644 (file)
--- a/modules/ldap/util_ldap.c
+++ b/modules/ldap/util_ldap.c
@@ -42,8 +42,6 @@
 #error mod_ldap requires httpd to detect LDAP support
 #endif
 
-#include "ldap_private.h"
-
 #ifdef LDAP_OPT_DEBUG_LEVEL
 #define AP_LDAP_OPT_DEBUG LDAP_OPT_DEBUG_LEVEL
 #else
@@ -2947,21 +2945,6 @@ static void util_ldap_register_hooks(apr_pool_t *p)
     APR_REGISTER_OPTIONAL_FN(uldap_ssl_supported);
     APR_REGISTER_OPTIONAL_FN(uldap_cache_check_subgroups);
 
-    APR_REGISTER_OPTIONAL_FN(ap_ldap_get_option);
-    APR_REGISTER_OPTIONAL_FN(ap_ldap_info);
-    APR_REGISTER_OPTIONAL_FN(ap_ldap_init);
-    APR_REGISTER_OPTIONAL_FN(ap_ldap_is_ldap_url);
-    APR_REGISTER_OPTIONAL_FN(ap_ldap_is_ldapi_url);
-    APR_REGISTER_OPTIONAL_FN(ap_ldap_is_ldaps_url);
-    APR_REGISTER_OPTIONAL_FN(ap_ldap_rebind_add);
-    APR_REGISTER_OPTIONAL_FN(ap_ldap_rebind_init);
-    APR_REGISTER_OPTIONAL_FN(ap_ldap_rebind_remove);
-    APR_REGISTER_OPTIONAL_FN(ap_ldap_set_option);
-    APR_REGISTER_OPTIONAL_FN(ap_ldap_ssl_deinit);
-    APR_REGISTER_OPTIONAL_FN(ap_ldap_ssl_init);
-    APR_REGISTER_OPTIONAL_FN(ap_ldap_url_parse);
-    APR_REGISTER_OPTIONAL_FN(ap_ldap_url_parse_ext);
-
     ap_hook_pre_config(util_ldap_pre_config, NULL, NULL, APR_HOOK_MIDDLE);
     ap_hook_post_config(util_ldap_post_config,NULL,NULL,APR_HOOK_MIDDLE);
     ap_hook_handler(util_ldap_handler, NULL, NULL, APR_HOOK_MIDDLE);