Migrated ldap plugin to INIT/METHOD macros

diff --git a/src/libstrongswan/plugins/ldap/ldap_fetcher.c b/src/libstrongswan/plugins/ldap/ldap_fetcher.c
index 59e655cd5df97273962ca080a6b18fcad7749992..e6c5922173bb2cb33184817bc553926fa490f173 100644 (file)
--- a/src/libstrongswan/plugins/ldap/ldap_fetcher.c
+++ b/src/libstrongswan/plugins/ldap/ldap_fetcher.c
@@ -100,8 +100,8 @@ static bool parse(LDAP *ldap, LDAPMessage *result, chunk_t *response)
 }
 
 
-static status_t fetch(private_ldap_fetcher_t *this, char *url,
-                                         chunk_t *result, va_list args)
+METHOD(fetcher_t, fetch, status_t,
+       private_ldap_fetcher_t *this, char *url, chunk_t *result)
 {
        LDAP *ldap;
        LDAPURLDesc *lurl;
@@ -166,10 +166,8 @@ static status_t fetch(private_ldap_fetcher_t *this, char *url,
 }
 
 
-/**
- * Implementation of fetcher_t.set_option.
- */
-static bool set_option(private_ldap_fetcher_t *this, fetcher_option_t option, ...)
+METHOD(fetcher_t, set_option, bool,
+       private_ldap_fetcher_t *this, fetcher_option_t option, ...)
 {
        va_list args;
 
@@ -186,10 +184,8 @@ static bool set_option(private_ldap_fetcher_t *this, fetcher_option_t option, ..
        }
 }
 
-/**
- * Implements ldap_fetcher_t.destroy
- */
-static void destroy(private_ldap_fetcher_t *this)
+METHOD(fetcher_t, destroy, void,
+       private_ldap_fetcher_t *this)
 {
        free(this);
 }
@@ -199,13 +195,18 @@ static void destroy(private_ldap_fetcher_t *this)
  */
 ldap_fetcher_t *ldap_fetcher_create()
 {
-       private_ldap_fetcher_t *this = malloc_thing(private_ldap_fetcher_t);
-
-       this->public.interface.fetch = (status_t(*)(fetcher_t*,char*,chunk_t*))fetch;
-       this->public.interface.set_option = (bool(*)(fetcher_t*, fetcher_option_t option, ...))set_option;
-       this->public.interface.destroy = (void (*)(fetcher_t*))destroy;
-
-       this->timeout = DEFAULT_TIMEOUT;
+       private_ldap_fetcher_t *this;
+
+       INIT(this,
+               .public = {
+                       .interface = {
+                               .fetch = _fetch,
+                               .set_option = _set_option,
+                               .destroy = _destroy,
+                       },
+               },
+               .timeout = DEFAULT_TIMEOUT,
+       );
 
        return &this->public;
 }

diff --git a/src/libstrongswan/plugins/ldap/ldap_plugin.c b/src/libstrongswan/plugins/ldap/ldap_plugin.c
index 372ac9f939cd1fe3156f37cafe58f8e3c1e91f26..3682ddd1ff8a2205b5b75e3fe60263c1eec0371b 100644 (file)
--- a/src/libstrongswan/plugins/ldap/ldap_plugin.c
+++ b/src/libstrongswan/plugins/ldap/ldap_plugin.c
@@ -31,10 +31,8 @@ struct private_ldap_plugin_t {
        ldap_plugin_t public;
 };
 
-/**
- * Implementation of ldap_plugin_t.destroy
- */
-static void destroy(private_ldap_plugin_t *this)
+METHOD(plugin_t, destroy, void,
+       private_ldap_plugin_t *this)
 {
        lib->fetcher->remove_fetcher(lib->fetcher,
                                                                 (fetcher_constructor_t)ldap_fetcher_create);
@@ -46,9 +44,15 @@ static void destroy(private_ldap_plugin_t *this)
  */
 plugin_t *ldap_plugin_create()
 {
-       private_ldap_plugin_t *this = malloc_thing(private_ldap_plugin_t);
+       private_ldap_plugin_t *this;
 
-       this->public.plugin.destroy = (void(*)(plugin_t*))destroy;
+       INIT(this,
+               .public = {
+                       .plugin = {
+                               .destroy = _destroy,
+                       },
+               },
+       );
 
        lib->fetcher->add_fetcher(lib->fetcher,
                                                (fetcher_constructor_t)ldap_fetcher_create, "ldap://");