Revert r1129886:

    Shift declaration logic of LDAP_DECLARE to ap_ldap.h from util_ldap.h

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

diff --git a/include/ap_ldap.h.in b/include/ap_ldap.h.in
index b847db3da37751551662fe1da7c3460882e9dbc1..e6d69af644c3315e3a506cdce767512dc474b0a3 100644 (file)
--- a/include/ap_ldap.h.in
+++ b/include/ap_ldap.h.in
@@ -30,27 +30,23 @@
  * @{
  */
 
-/* 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)
+#define MODLDAP_DECLARE(type)            type
+#define MODLDAP_DECLARE_NONSTD(type)     type
+#define MODLDAP_DECLARE_DATA
+#elif defined(MODLDAP_DECLARE_STATIC)
+#define MODLDAP_DECLARE(type)            type __stdcall
+#define MODLDAP_DECLARE_NONSTD(type)     type
+#define MODLDAP_DECLARE_DATA
+#elif defined(MODLDAP_DECLARE_EXPORT)
+#define MODLDAP_DECLARE(type)            __declspec(dllexport) type __stdcall
+#define MODLDAP_DECLARE_NONSTD(type)     __declspec(dllexport) type
+#define MODLDAP_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)
+#define MODLDAP_DECLARE(type)            __declspec(dllimport) type __stdcall
+#define MODLDAP_DECLARE_NONSTD(type)     __declspec(dllimport) type
+#define MODLDAP_DECLARE_DATA             __declspec(dllimport)
 #endif
-
 /* this will be defined if LDAP support was compiled into apr-util */
 #define AP_HAS_LDAP              @ap_has_ldap@
 

diff --git a/include/ap_ldap_init.h b/include/ap_ldap_init.h
index ce2ae2c0a6f3b5f392607f6a47366c9730042680..799eedd977d29cb07330c37940044c9461d5c9a3 100644 (file)
--- a/include/ap_ldap_init.h
+++ b/include/ap_ldap_init.h
@@ -64,7 +64,7 @@ extern "C" {
 
 
 /**
- * LDAP SSL Initialise function
+ * MODLDAP SSL Initialise function
  *
  * This function initialises SSL on the underlying LDAP toolkit
  * if this is necessary.
@@ -86,13 +86,13 @@ extern "C" {
  * ap_ldap_set_option() AP_LDAP_OPT_TLS_CERT option for details.
  * @param result_err The returned result
  */
-LDAP_DECLARE(int) ap_ldap_ssl_init(apr_pool_t *pool,
+MODLDAP_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
+ * MODLDAP SSL De-Initialise function
  *
  * This function tears down any SSL certificate setup previously
  * set using ap_ldap_ssl_init(). It should be called to clean
@@ -100,10 +100,10 @@ LDAP_DECLARE(int) ap_ldap_ssl_init(apr_pool_t *pool,
  * @todo currently we do not check whether ap_ldap_ssl_init()
  * has been called first - we probably should.
  */
-LDAP_DECLARE(int) ap_ldap_ssl_deinit(void);
+MODLDAP_DECLARE(int) ap_ldap_ssl_deinit(void);
 
 /**
- * LDAP initialise function
+ * MODLDAP initialise function
  *
  * This function is responsible for initialising an LDAP
  * connection in a toolkit independant way. It does the
@@ -136,7 +136,7 @@ LDAP_DECLARE(int) ap_ldap_ssl_deinit(void);
  * @param secure The security mode to set
  * @param result_err The returned result
  */
-LDAP_DECLARE(int) ap_ldap_init(apr_pool_t *pool,
+MODLDAP_DECLARE(int) ap_ldap_init(apr_pool_t *pool,
                                   LDAP **ldap,
                                   const char *hostname,
                                   int portno,
@@ -144,14 +144,14 @@ LDAP_DECLARE(int) ap_ldap_init(apr_pool_t *pool,
                                   ap_ldap_err_t **result_err);
 
 /**
- * LDAP info function
+ * MODLDAP info function
  *
  * This function returns a string describing the LDAP toolkit
  * currently in use. The string is placed inside result_err->reason.
  * @param pool The pool to use
  * @param result_err The returned result
  */
-LDAP_DECLARE(int) ap_ldap_info(apr_pool_t *pool,
+MODLDAP_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 9c4898a2251cc225cca48e7a85a9ccb96ad414e9..ba6d9958920bb8e3fae8457eb156c9e6ae944405 100644 (file)
--- a/include/ap_ldap_option.h
+++ b/include/ap_ldap_option.h
@@ -16,7 +16,7 @@
 
 /**
  * @file ap_ldap_option.h
- * @brief LDAP ldap_*_option() functions
+ * @brief MODLDAP ldap_*_option() functions
  */
 #ifndef AP_LDAP_OPTION_H
 #define AP_LDAP_OPTION_H
@@ -203,7 +203,7 @@ struct ap_ldap_opt_tls_cert_t {
 #define AP_LDAP_STOPTLS 3
 
 /**
- * LDAP get option function
+ * MODLDAP get option function
  *
  * This function gets option values from a given LDAP session if
  * one was specified. It maps to the native ldap_get_option() function.
@@ -214,14 +214,14 @@ struct ap_ldap_opt_tls_cert_t {
  * @param result_err The ap_ldap_err_t structure contained detailed results
  *        of the operation.
  */
-LDAP_DECLARE(int) ap_ldap_get_option(apr_pool_t *pool,
+MODLDAP_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
+ * MODLDAP set option function
  * 
  * This function sets option values to a given LDAP session if
  * one was specified. It maps to the native ldap_set_option() function.
@@ -236,7 +236,7 @@ LDAP_DECLARE(int) ap_ldap_get_option(apr_pool_t *pool,
  * @param result_err The ap_ldap_err_t structure contained detailed results
  *        of the operation.
  */
-LDAP_DECLARE(int) ap_ldap_set_option(apr_pool_t *pool,
+MODLDAP_DECLARE(int) ap_ldap_set_option(apr_pool_t *pool,
                                         LDAP *ldap,
                                         int option,
                                         const void *invalue,

diff --git a/include/ap_ldap_rebind.h b/include/ap_ldap_rebind.h
index 5601a861eab7a38d89dd3785310d45aa4ed647c5..91c30e380d33de7d0d60d69ac01daa84bbefc666 100644 (file)
--- a/include/ap_ldap_rebind.h
+++ b/include/ap_ldap_rebind.h
@@ -15,7 +15,7 @@
  */
 
 /**
- * The LDAP rebind functions provide an implementation of
+ * The MODLDAP rebind functions provide an implementation of
  * a rebind procedure that can be used to allow clients to chase referrals,
  * using the same credentials used to log in originally.
  *
@@ -43,16 +43,16 @@
 #if AP_HAS_LDAP
 
 /**
- * LDAP initialize rebind lock
+ * MODLDAP initialize rebind lock
  *
  * This function creates the lock for controlling access to the xref list..
  * @param pool Pool to use when creating the xref_lock.
  */
-LDAP_DECLARE(apr_status_t) ap_ldap_rebind_init(apr_pool_t *pool);
+MODLDAP_DECLARE(apr_status_t) ap_ldap_rebind_init(apr_pool_t *pool);
 
 
 /**
- * LDAP rebind_add function
+ * MODLDAP rebind_add function
  *
  * This function creates a cross reference entry for the specified ldap
  * connection. The rebind callback function will look up this ldap 
@@ -72,13 +72,13 @@ LDAP_DECLARE(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.
  */
-LDAP_DECLARE(apr_status_t) ap_ldap_rebind_add(apr_pool_t *pool,
+MODLDAP_DECLARE(apr_status_t) ap_ldap_rebind_add(apr_pool_t *pool,
                                                  LDAP *ld,
                                                  const char *bindDN,
                                                  const char *bindPW);
 
 /**
- * LDAP rebind_remove function
+ * MODLDAP rebind_remove function
  *
  * This function removes the rebind cross reference entry for the
  * specified ldap connection.
@@ -88,7 +88,7 @@ LDAP_DECLARE(apr_status_t) ap_ldap_rebind_add(apr_pool_t *pool,
  *
  * @param ld The LDAP connectionhandle
  */
-LDAP_DECLARE(apr_status_t) ap_ldap_rebind_remove(LDAP *ld);
+MODLDAP_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 d8c409582264cbb7a72b3d86daf1f80db7a28be3..9f2bca1d0890378410776dd3b0fe7c7e74583829 100644 (file)
--- a/include/ap_ldap_url.h
+++ b/include/ap_ldap_url.h
@@ -16,7 +16,7 @@
 
 /**
  * @file ap_ldap_url.h
- * @brief LDAP ldap_init() functions
+ * @brief MODLDAP ldap_init() functions
  */
 #ifndef AP_LDAP_URL_H
 #define AP_LDAP_URL_H
@@ -71,19 +71,19 @@ typedef struct ap_ldap_url_desc_t {
  * Is this URL an ldap url? ldap://
  * @param url The url to test
  */
-LDAP_DECLARE(int) ap_ldap_is_ldap_url(const char *url);
+MODLDAP_DECLARE(int) ap_ldap_is_ldap_url(const char *url);
 
 /**
  * Is this URL an SSL ldap url? ldaps://
  * @param url The url to test
  */
-LDAP_DECLARE(int) ap_ldap_is_ldaps_url(const char *url);
+MODLDAP_DECLARE(int) ap_ldap_is_ldaps_url(const char *url);
 
 /**
  * Is this URL an ldap socket url? ldapi://
  * @param url The url to test
  */
-LDAP_DECLARE(int) ap_ldap_is_ldapi_url(const char *url);
+MODLDAP_DECLARE(int) ap_ldap_is_ldapi_url(const char *url);
 
 /**
  * Parse an LDAP URL.
@@ -92,7 +92,7 @@ LDAP_DECLARE(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
  */
-LDAP_DECLARE(int) ap_ldap_url_parse_ext(apr_pool_t *pool,
+MODLDAP_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);
@@ -104,7 +104,7 @@ LDAP_DECLARE(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
  */
-LDAP_DECLARE(int) ap_ldap_url_parse(apr_pool_t *pool,
+MODLDAP_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);

diff --git a/include/util_ldap.h b/include/util_ldap.h
index 5ee3fb604347bb1d596ba0d4d7b68dbcfc59058d..2ab4b7eb3f4e2c62262f497a4bac4367ae54c626 100644 (file)
--- a/include/util_ldap.h
+++ b/include/util_ldap.h
@@ -56,6 +56,27 @@
 #include "http_request.h"
 #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
+
 #ifdef __cplusplus
 extern "C" {
 #endif

diff --git a/modules/ldap/ap_ldap_init.c b/modules/ldap/ap_ldap_init.c
index 480229025129dcc8396bcbe5835b8c8c3dec814d..ec34904e46084705b609867922e782957aa4f3bf 100644 (file)
--- a/modules/ldap/ap_ldap_init.c
+++ b/modules/ldap/ap_ldap_init.c
@@ -51,7 +51,7 @@
  * will return APR_EGENERAL. Further LDAP specific error information
  * can be found in result_err.
  */
-LDAP_DECLARE(int) ap_ldap_ssl_init(apr_pool_t *pool,
+MODLDAP_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)
@@ -108,7 +108,7 @@ LDAP_DECLARE(int) ap_ldap_ssl_init(apr_pool_t *pool,
  * @todo currently we do not check whether ap_ldap_ssl_init()
  * has been called first - should we?
  */
-LDAP_DECLARE(int) ap_ldap_ssl_deinit(void)
+MODLDAP_DECLARE(int) ap_ldap_ssl_deinit(void)
 {
 
 #if AP_HAS_LDAP_SSL && AP_HAS_LDAPSSL_CLIENT_DEINIT
@@ -139,7 +139,7 @@ LDAP_DECLARE(int) ap_ldap_ssl_deinit(void)
  * AP_LDAP_SSL: SSL encryption (ldaps://)
  * AP_LDAP_STARTTLS: Force STARTTLS on ldap://
  */
-LDAP_DECLARE(int) ap_ldap_init(apr_pool_t *pool,
+MODLDAP_DECLARE(int) ap_ldap_init(apr_pool_t *pool,
                                   LDAP **ldap,
                                   const char *hostname,
                                   int portno,
@@ -179,7 +179,7 @@ LDAP_DECLARE(int) ap_ldap_init(apr_pool_t *pool,
  * This function returns a string describing the LDAP toolkit
  * currently in use. The string is placed inside result_err->reason.
  */
-LDAP_DECLARE(int) ap_ldap_info(apr_pool_t *pool,
+MODLDAP_DECLARE(int) ap_ldap_info(apr_pool_t *pool,
                                   ap_ldap_err_t **result_err)
 {
     ap_ldap_err_t *result = (ap_ldap_err_t *)apr_pcalloc(pool, sizeof(ap_ldap_err_t));

diff --git a/modules/ldap/ap_ldap_option.c b/modules/ldap/ap_ldap_option.c
index d38ad21b07d92dfa7bbb2628e3e915ef4fca7723..84e770dc4b42c527c4c087091f46e21f5f61e5f0 100644 (file)
--- a/modules/ldap/ap_ldap_option.c
+++ b/modules/ldap/ap_ldap_option.c
@@ -38,12 +38,12 @@ static void option_set_tls(apr_pool_t *pool, LDAP *ldap, const void *invalue,
                            ap_ldap_err_t *result);
 
 /**
- * LDAP get option function
+ * MODLDAP get option function
  *
  * This function gets option values from a given LDAP session if
  * one was specified.
  */
-LDAP_DECLARE(int) ap_ldap_get_option(apr_pool_t *pool,
+MODLDAP_DECLARE(int) ap_ldap_get_option(apr_pool_t *pool,
                                         LDAP *ldap,
                                         int option,
                                         void *outvalue,
@@ -72,7 +72,7 @@ LDAP_DECLARE(int) ap_ldap_get_option(apr_pool_t *pool,
 } 
 
 /**
- * LDAP set option function
+ * MODLDAP set option function
  *
  * This function sets option values to a given LDAP session if
  * one was specified.
@@ -81,7 +81,7 @@ LDAP_DECLARE(int) ap_ldap_get_option(apr_pool_t *pool,
  * will try and apply legacy functions to achieve the same effect,
  * depending on the platform.
  */
-LDAP_DECLARE(int) ap_ldap_set_option(apr_pool_t *pool,
+MODLDAP_DECLARE(int) ap_ldap_set_option(apr_pool_t *pool,
                                         LDAP *ldap,
                                         int option,
                                         const void *invalue,

diff --git a/modules/ldap/ap_ldap_rebind.c b/modules/ldap/ap_ldap_rebind.c
index 230986ef6bed4768e844dfa754e77c2920fa4703..0b93abc9d9213cb97d727af1269bf87736ace9cd 100644 (file)
--- a/modules/ldap/ap_ldap_rebind.c
+++ b/modules/ldap/ap_ldap_rebind.c
@@ -67,7 +67,7 @@ static apr_status_t ap_ldap_pool_cleanup_set_null(void *data_)
 
 
 /* AP utility routine used to create the xref_lock. */
-LDAP_DECLARE(apr_status_t) ap_ldap_rebind_init(apr_pool_t *pool)
+MODLDAP_DECLARE(apr_status_t) ap_ldap_rebind_init(apr_pool_t *pool)
 {
     apr_status_t retcode = APR_SUCCESS;
 
@@ -89,7 +89,7 @@ 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,
+MODLDAP_DECLARE(apr_status_t) ap_ldap_rebind_add(apr_pool_t *pool,
                                                    LDAP *ld, 
                                                    const char *bindDN, 
                                                    const char *bindPW)
@@ -147,7 +147,7 @@ LDAP_DECLARE(apr_status_t) ap_ldap_rebind_add(apr_pool_t *pool,
 }
 
 
-LDAP_DECLARE(apr_status_t) ap_ldap_rebind_remove(LDAP *ld)
+MODLDAP_DECLARE(apr_status_t) ap_ldap_rebind_remove(LDAP *ld)
 {
     ap_ldap_rebind_entry_t *tmp_xref, *prev = NULL;
     apr_status_t retcode = 0;

diff --git a/modules/ldap/ap_ldap_url.c b/modules/ldap/ap_ldap_url.c
index 8317990c7562cb34d5ae255aadc31590c607df1f..f1fe2166ee4b86efc909bd32bf550369c93b4f66 100644 (file)
--- a/modules/ldap/ap_ldap_url.c
+++ b/modules/ldap/ap_ldap_url.c
@@ -115,7 +115,7 @@ static char **ap_ldap_str2charray(apr_pool_t *pool,
  * Is this URL an ldap url?
  *
  */
-LDAP_DECLARE(int) ap_ldap_is_ldap_url(const char *url)
+MODLDAP_DECLARE(int) ap_ldap_is_ldap_url(const char *url)
 {
     int enclosed;
     const char * scheme;
@@ -135,7 +135,7 @@ LDAP_DECLARE(int) ap_ldap_is_ldap_url(const char *url)
  * Is this URL a secure ldap url?
  *
  */
-LDAP_DECLARE(int) ap_ldap_is_ldaps_url(const char *url)
+MODLDAP_DECLARE(int) ap_ldap_is_ldaps_url(const char *url)
 {
     int enclosed;
     const char * scheme;
@@ -155,7 +155,7 @@ LDAP_DECLARE(int) ap_ldap_is_ldaps_url(const char *url)
  * Is this URL an ldap socket url?
  *
  */
-LDAP_DECLARE(int) ap_ldap_is_ldapi_url(const char *url)
+MODLDAP_DECLARE(int) ap_ldap_is_ldapi_url(const char *url)
 {
     int enclosed;
     const char * scheme;
@@ -257,7 +257,7 @@ static int str2scope(const char *p)
  * The LDAP result code and reason string is returned in the
  * ap_ldap_err_t structure.
  */
-LDAP_DECLARE(int) ap_ldap_url_parse_ext(apr_pool_t *pool,
+MODLDAP_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)
@@ -580,7 +580,7 @@ LDAP_DECLARE(int) ap_ldap_url_parse_ext(apr_pool_t *pool,
  * The LDAP result code and reason string is returned in the
  * ap_ldap_err_t structure.
  */
-LDAP_DECLARE(int) ap_ldap_url_parse(apr_pool_t *pool,
+MODLDAP_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)