Fixed a few memory leaks in LDAP search results handling.
[claws.git] / src / ldapctrl.c
index 2b4b65effa02291fbafcdfce66bc392c47b06ea4..62adf5b043b8703a4a0c2b8438775d3564092f03 100644 (file)
@@ -1,10 +1,10 @@
 /*
  * Sylpheed -- a GTK+ based, lightweight, and fast e-mail client
- * Copyright (C) 2003-2004 Match Grun
+ * Copyright (C) 2003-2012 Match Grun and the Claws Mail team
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
+ * the Free Software Foundation; either version 3 of the License, or
  * (at your option) any later version.
  *
  * This program is distributed in the hope that it will be useful,
@@ -13,8 +13,8 @@
  * GNU General Public License for more details.
  *
  * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
+ * 
  */
 
 /*
@@ -23,6 +23,7 @@
 
 #ifdef HAVE_CONFIG_H
 #  include "config.h"
+#include "claws-features.h"
 #endif
 
 #ifdef USE_LDAP
 
 #include "ldapctrl.h"
 #include "mgutils.h"
+#include "passwordstore.h"
+#include "editaddress_other_attributes_ldap.h"
+#include "common/utils.h"
+#include "common/quoted-printable.h"
 
 /**
  * Create new LDAP control block object.
@@ -46,18 +51,19 @@ LdapControl *ldapctl_create( void ) {
        ctl->port = LDAPCTL_DFL_PORT;
        ctl->baseDN = NULL;
        ctl->bindDN = NULL;
-       ctl->bindPass = NULL;
        ctl->listCriteria = NULL;
        ctl->attribEMail = g_strdup( LDAPCTL_ATTR_EMAIL );
        ctl->attribCName = g_strdup( LDAPCTL_ATTR_COMMONNAME );
        ctl->attribFName = g_strdup( LDAPCTL_ATTR_GIVENNAME );
        ctl->attribLName = g_strdup( LDAPCTL_ATTR_SURNAME );
+       ctl->attribDName = g_strdup( LDAPCTL_ATTR_DISPLAYNAME );
        ctl->maxEntries = LDAPCTL_MAX_ENTRIES;
        ctl->timeOut = LDAPCTL_DFL_TIMEOUT;
        ctl->maxQueryAge = LDAPCTL_DFL_QUERY_AGE;
        ctl->matchingOption = LDAPCTL_MATCH_BEGINWITH;
        ctl->version = 0;
        ctl->enableTLS = FALSE;
+       ctl->enableSSL = FALSE;
 
        /* Mutex to protect control block */
        ctl->mutexCtl = g_malloc0( sizeof( pthread_mutex_t ) );
@@ -73,7 +79,12 @@ LdapControl *ldapctl_create( void ) {
  */
 void ldapctl_set_host( LdapControl* ctl, const gchar *value ) {
        ctl->hostName = mgu_replace_string( ctl->hostName, value );
+
+       if ( ctl->hostName == NULL )
+               return;
+
        g_strstrip( ctl->hostName );
+       debug_print("setting hostname: %s\n", ctl->hostName);
 }
 
 /**
@@ -88,6 +99,7 @@ void ldapctl_set_port( LdapControl* ctl, const gint value ) {
        else {
                ctl->port = LDAPCTL_DFL_PORT;
        }
+       debug_print("setting port: %d\n", ctl->port);
 }
 
 /**
@@ -97,7 +109,12 @@ void ldapctl_set_port( LdapControl* ctl, const gint value ) {
  */
 void ldapctl_set_base_dn( LdapControl* ctl, const gchar *value ) {
        ctl->baseDN = mgu_replace_string( ctl->baseDN, value );
+
+       if ( ctl->baseDN == NULL )
+               return;
+
        g_strstrip( ctl->baseDN );
+       debug_print("setting baseDN: %s\n", ctl->baseDN);
 }
 
 /**
@@ -107,17 +124,12 @@ void ldapctl_set_base_dn( LdapControl* ctl, const gchar *value ) {
  */
 void ldapctl_set_bind_dn( LdapControl* ctl, const gchar *value ) {
        ctl->bindDN = mgu_replace_string( ctl->bindDN, value );
-       g_strstrip( ctl->bindDN );
-}
 
-/**
- * Specify bind password to be used.
- * \param ctl  Control object to process.
- * \param value Password.
- */
-void ldapctl_set_bind_password( LdapControl* ctl, const gchar *value ) {
-       ctl->bindPass = mgu_replace_string( ctl->bindPass, value );
-       g_strstrip( ctl->bindPass );
+       if ( ctl->bindDN == NULL )
+               return;
+
+       g_strstrip( ctl->bindDN );
+       debug_print("setting bindDN: %s\n", ctl->bindDN);
 }
 
 /**
@@ -132,6 +144,7 @@ void ldapctl_set_max_entries( LdapControl* ctl, const gint value ) {
        else {
                ctl->maxEntries = LDAPCTL_MAX_ENTRIES;
        }
+       debug_print("setting maxEntries: %d\n", ctl->maxEntries);
 }
 
 /**
@@ -146,6 +159,7 @@ void ldapctl_set_timeout( LdapControl* ctl, const gint value ) {
        else {
                ctl->timeOut = LDAPCTL_DFL_TIMEOUT;
        }
+       debug_print("setting timeOut: %d\n", ctl->timeOut);
 }
 
 /**
@@ -163,6 +177,7 @@ void ldapctl_set_max_query_age( LdapControl* ctl, const gint value ) {
        else {
                ctl->maxQueryAge = value;
        }
+       debug_print("setting maxAge: %d\n", ctl->maxQueryAge);
 }
 
 /**
@@ -184,6 +199,7 @@ void ldapctl_set_matching_option( LdapControl* ctl, const gint value ) {
        else {
                ctl->matchingOption = value;
        }
+       debug_print("setting matchingOption: %d\n", ctl->matchingOption);
 }
 
 /**
@@ -192,18 +208,17 @@ void ldapctl_set_matching_option( LdapControl* ctl, const gint value ) {
  * \param value <i>TRUE</i> to enable TLS.
  */
 void ldapctl_set_tls( LdapControl* ctl, const gboolean value ) {
+#if (defined USE_LDAP_TLS || defined G_OS_WIN32)
        ctl->enableTLS = value;
+       debug_print("setting STARTTLS: %d\n", ctl->enableTLS);
+#endif
 }
 
-/**
- * Specify search criteria list to be used.
- * \param ctl   Control data object.
- * \param value Linked list of LDAP attribute names to use for search.
- */
-void ldapctl_set_criteria_list( LdapControl* ctl, GList *value ) {
-       g_return_if_fail( ctl != NULL );
-       mgu_free_dlist( ctl->listCriteria );
-       ctl->listCriteria = value;
+void ldapctl_set_ssl( LdapControl* ctl, const gboolean value ) {
+#if (defined USE_LDAP_TLS || defined G_OS_WIN32)
+       ctl->enableSSL = value;
+       debug_print("setting SSL/TLS: %d\n", ctl->enableSSL);
+#endif
 }
 
 /**
@@ -216,7 +231,7 @@ void ldapctl_set_criteria_list( LdapControl* ctl, GList *value ) {
  *         <code>ldapctl_criteria_list_add()</code> functions for this purpose.
  */
 GList *ldapctl_get_criteria_list( const LdapControl* ctl ) {
-       g_return_val_if_fail( ctl != NULL, NULL );
+       cm_return_val_if_fail( ctl != NULL, NULL );
        return ctl->listCriteria;
 }
 
@@ -225,7 +240,7 @@ GList *ldapctl_get_criteria_list( const LdapControl* ctl ) {
  * \param  ctl  Control data object.
  */
 void ldapctl_criteria_list_clear( LdapControl *ctl ) {
-       g_return_if_fail( ctl != NULL );
+       cm_return_if_fail( ctl != NULL );
        mgu_free_dlist( ctl->listCriteria );
        ctl->listCriteria = NULL;
 }
@@ -237,45 +252,33 @@ void ldapctl_criteria_list_clear( LdapControl *ctl ) {
  *             be appended to the list.
  */
 void ldapctl_criteria_list_add( LdapControl *ctl, gchar *attr ) {
-       g_return_if_fail( ctl != NULL );
+       cm_return_if_fail( ctl != NULL );
        if( attr != NULL ) {
                if( mgu_list_test_unq_nc( ctl->listCriteria, attr ) ) {
+                       debug_print("adding to criteria list: %s\n", attr);
                        ctl->listCriteria = g_list_append(
                                ctl->listCriteria, g_strdup( attr ) );
                }
        }
 }
 
-/**
- * Build criteria list using default attributes.
- * \param ctl  Control object to process.
- */
-void ldapctl_default_attributes( LdapControl *ctl ) {
-       g_return_if_fail( ctl != NULL );
-
-       ldapctl_criteria_list_clear( ctl );
-       ldapctl_criteria_list_add( ctl, LDAPCTL_ATTR_COMMONNAME );
-       ldapctl_criteria_list_add( ctl, LDAPCTL_ATTR_GIVENNAME );
-       ldapctl_criteria_list_add( ctl, LDAPCTL_ATTR_SURNAME );
-       ldapctl_criteria_list_add( ctl, LDAPCTL_ATTR_EMAIL );
-}
-
 /**
  * Clear LDAP server member variables.
  * \param ctl Control object to clear.
  */
-void ldapctl_clear( LdapControl *ctl ) {
-       g_return_if_fail( ctl != NULL );
+static void ldapctl_clear( LdapControl *ctl ) {
+       cm_return_if_fail( ctl != NULL );
 
+       debug_print("clearing ldap controller members\n");
        /* Free internal stuff */
        g_free( ctl->hostName );
        g_free( ctl->baseDN );
        g_free( ctl->bindDN );
-       g_free( ctl->bindPass );
        g_free( ctl->attribEMail );
        g_free( ctl->attribCName );
        g_free( ctl->attribFName );
        g_free( ctl->attribLName );
+       g_free( ctl->attribDName );
 
        ldapctl_criteria_list_clear( ctl );
 
@@ -284,17 +287,18 @@ void ldapctl_clear( LdapControl *ctl ) {
        ctl->port = 0;
        ctl->baseDN = NULL;
        ctl->bindDN = NULL;
-       ctl->bindPass = NULL;
        ctl->attribEMail = NULL;
        ctl->attribCName = NULL;
        ctl->attribFName = NULL;
        ctl->attribLName = NULL;
+       ctl->attribDName = NULL;
        ctl->maxEntries = 0;
        ctl->timeOut = 0;
        ctl->maxQueryAge = 0;
        ctl->matchingOption = LDAPCTL_MATCH_BEGINWITH;
        ctl->version = 0;
        ctl->enableTLS = FALSE;
+       ctl->enableSSL = FALSE;
 }
 
 /**
@@ -302,8 +306,9 @@ void ldapctl_clear( LdapControl *ctl ) {
  * \param ctl Control object to free.
  */
 void ldapctl_free( LdapControl *ctl ) {
-       g_return_if_fail( ctl != NULL );
+       cm_return_if_fail( ctl != NULL );
 
+       debug_print("releasing requested memory for ldap controller\n");
        /* Free internal stuff */
        ldapctl_clear( ctl );
 
@@ -316,57 +321,38 @@ void ldapctl_free( LdapControl *ctl ) {
        g_free( ctl );
 }
 
-/**
- * Setup default (empty) values for specified object.
- * \param ctl  Control object to process.
- */
-void ldapctl_default_values( LdapControl *ctl ) {
-       g_return_if_fail( ctl != NULL );
-
-       /* Clear our destination */
-       ldapctl_clear( ctl );
-
-       /* Copy strings */
-       ctl->hostName = g_strdup( "" );
-       ctl->baseDN = g_strdup( "" );
-       ctl->bindDN = g_strdup( "" );
-       ctl->bindPass = g_strdup( "" );
-       ctl->port = LDAPCTL_DFL_PORT;
-       ctl->maxEntries = LDAPCTL_MAX_ENTRIES;
-       ctl->timeOut = LDAPCTL_DFL_TIMEOUT;
-       ctl->maxQueryAge = LDAPCTL_DFL_QUERY_AGE;
-       ctl->matchingOption = LDAPCTL_MATCH_BEGINWITH;
-       ctl->version = 0;
-       ctl->enableTLS = FALSE;
-
-       ldapctl_default_attributes( ctl );
-}
-
 /**
  * Display object to specified stream.
  * \param ctl    Control object to process.
  * \param stream Output stream.
  */
 void ldapctl_print( const LdapControl *ctl, FILE *stream ) {
-       g_return_if_fail( ctl != NULL );
+       cm_return_if_fail( ctl != NULL );
+       gchar *pwd;
 
        pthread_mutex_lock( ctl->mutexCtl );
        fprintf( stream, "LdapControl:\n" );
-       fprintf( stream, "host name: '%s'\n", ctl->hostName );
+       fprintf( stream, "host name: '%s'\n", ctl->hostName?ctl->hostName:"null" );
        fprintf( stream, "     port: %d\n",   ctl->port );
-       fprintf( stream, "  base dn: '%s'\n", ctl->baseDN );
-       fprintf( stream, "  bind dn: '%s'\n", ctl->bindDN );
-       fprintf( stream, "bind pass: '%s'\n", ctl->bindPass );
-       fprintf( stream, "attr mail: '%s'\n", ctl->attribEMail );
-       fprintf( stream, "attr comn: '%s'\n", ctl->attribCName );
-       fprintf( stream, "attr frst: '%s'\n", ctl->attribFName );
-       fprintf( stream, "attr last: '%s'\n", ctl->attribLName );
+       fprintf( stream, "  base dn: '%s'\n", ctl->baseDN?ctl->baseDN:"null" );
+       fprintf( stream, "  bind dn: '%s'\n", ctl->bindDN?ctl->bindDN:"null" );
+       pwd = passwd_store_get(PWS_CORE, "LDAP", ctl->hostName);
+       fprintf( stream, "bind pass: '%s'\n", pwd?pwd:"null" );
+       if (pwd != NULL && strlen(pwd) > 0)
+               memset(pwd, 0, strlen(pwd));
+       g_free(pwd);
+       fprintf( stream, "attr mail: '%s'\n", ctl->attribEMail?ctl->attribEMail:"null" );
+       fprintf( stream, "attr comn: '%s'\n", ctl->attribCName?ctl->attribCName:"null" );
+       fprintf( stream, "attr frst: '%s'\n", ctl->attribFName?ctl->attribFName:"null" );
+       fprintf( stream, "attr last: '%s'\n", ctl->attribLName?ctl->attribLName:"null" );
+       fprintf( stream, "attr disn: '%s'\n", ctl->attribDName?ctl->attribDName:"null" );
        fprintf( stream, "max entry: %d\n",   ctl->maxEntries );
        fprintf( stream, "  timeout: %d\n",   ctl->timeOut );
        fprintf( stream, "  max age: %d\n",   ctl->maxQueryAge );
        fprintf( stream, "match opt: %d\n",   ctl->matchingOption );
        fprintf( stream, "  version: %d\n",   ctl->version );
-       fprintf( stream, "      TLS: %s\n",   ctl->enableTLS ? "yes" : "no" );
+       fprintf( stream, " STARTTLS: %s\n",   ctl->enableTLS ? "yes" : "no" );
+       fprintf( stream, "  SSL/TLS: %s\n",   ctl->enableSSL ? "yes" : "no" );
        fprintf( stream, "crit list:\n" );
        if( ctl->listCriteria ) {
                mgu_print_dlist( ctl->listCriteria, stream );
@@ -386,9 +372,10 @@ void ldapctl_print( const LdapControl *ctl, FILE *stream ) {
 void ldapctl_copy( const LdapControl *ctlFrom, LdapControl *ctlTo ) {
        GList *node;
 
-       g_return_if_fail( ctlFrom != NULL );
-       g_return_if_fail( ctlTo != NULL );
+       cm_return_if_fail( ctlFrom != NULL );
+       cm_return_if_fail( ctlTo != NULL );
 
+       debug_print("ldap controller copy\n");
        /* Lock both objects */
        pthread_mutex_lock( ctlFrom->mutexCtl );
        pthread_mutex_lock( ctlTo->mutexCtl );
@@ -400,11 +387,11 @@ void ldapctl_copy( const LdapControl *ctlFrom, LdapControl *ctlTo ) {
        ctlTo->hostName = g_strdup( ctlFrom->hostName );
        ctlTo->baseDN = g_strdup( ctlFrom->baseDN );
        ctlTo->bindDN = g_strdup( ctlFrom->bindDN );
-       ctlTo->bindPass = g_strdup( ctlFrom->bindPass );
        ctlTo->attribEMail = g_strdup( ctlFrom->attribEMail );
        ctlTo->attribCName = g_strdup( ctlFrom->attribCName );
        ctlTo->attribFName = g_strdup( ctlFrom->attribFName );
        ctlTo->attribLName = g_strdup( ctlFrom->attribLName );
+       ctlTo->attribDName = g_strdup( ctlFrom->attribDName );
 
        /* Copy search criteria */
        node = ctlFrom->listCriteria;
@@ -422,6 +409,7 @@ void ldapctl_copy( const LdapControl *ctlFrom, LdapControl *ctlTo ) {
        ctlTo->matchingOption = ctlFrom->matchingOption;
        ctlTo->version = ctlFrom->version;
        ctlTo->enableTLS = ctlFrom->enableTLS;
+       ctlTo->enableSSL = ctlFrom->enableSSL;
 
        /* Unlock */
        pthread_mutex_unlock( ctlTo->mutexCtl );
@@ -500,6 +488,7 @@ static gchar *ldapctl_build_ldap_criteria(
                g_free( p2 );
        }
        g_free( term );
+       debug_print("search criteria: %s\n", crit?crit:"null");
        return crit;
 }
 
@@ -525,8 +514,8 @@ gchar *ldapctl_format_criteria( LdapControl *ctl, const gchar *searchVal ) {
        gchar *p1, *p2, *retVal;
        gchar *criteriaFmt;
 
-       g_return_val_if_fail( ctl != NULL, NULL );
-       g_return_val_if_fail( searchVal != NULL, NULL );
+       cm_return_val_if_fail( ctl != NULL, NULL );
+       cm_return_val_if_fail( searchVal != NULL, NULL );
 
        /* Test whether there are more that one search terms */
        retVal = ldapctl_build_ldap_criteria( searchVal, ctl->matchingOption );
@@ -546,7 +535,6 @@ gchar *ldapctl_format_criteria( LdapControl *ctl, const gchar *searchVal ) {
        node = ctl->listCriteria;
        while( node ) {
                gchar *attr, *tmp;
-
                attr = node->data;
                node = g_list_next( node );
 
@@ -557,8 +545,15 @@ gchar *ldapctl_format_criteria( LdapControl *ctl, const gchar *searchVal ) {
                        /* Subsequent time through */
                        gchar *crit;
 
-                       /* Format query criteria */
-                       crit = g_strdup_printf( criteriaFmt, attr, searchVal );
+                       debug_print("crit: %s\n", searchVal);
+                       /* fix bug when doing a search any */
+                       if (strcmp("*@", searchVal) == 0) {
+                           crit = g_strdup_printf( "(%s=*)", attr );
+                       }
+                       else {
+                           /* Format query criteria */
+                           crit = g_strdup_printf( criteriaFmt, attr, searchVal );
+                       }
 
                        /* Append to existing criteria */                       
                        g_free( p2 );
@@ -568,7 +563,13 @@ gchar *ldapctl_format_criteria( LdapControl *ctl, const gchar *searchVal ) {
                }
                else {
                        /* First time through - Format query criteria */
-                       p2 = g_strdup_printf( criteriaFmt, attr, searchVal );
+                        /* fix bug when doing a search any */
+                       if (strcmp("*@", searchVal) == 0) {
+                           p2 = g_strdup_printf( "(%s=*)", attr );
+                       }
+                       else {
+                           p2 = g_strdup_printf( criteriaFmt, attr, searchVal );
+                       }
                }
        }
 
@@ -581,6 +582,7 @@ gchar *ldapctl_format_criteria( LdapControl *ctl, const gchar *searchVal ) {
                retVal = p2;
                g_free( p1 );
        }
+       debug_print("current search string: %s\n", retVal);
        return retVal;
 }
 
@@ -593,14 +595,59 @@ char **ldapctl_attribute_array( LdapControl *ctl ) {
        char **ptrArray;
        GList *node;
        gint cnt, i;
-       g_return_val_if_fail( ctl != NULL, NULL );
+       cm_return_val_if_fail( ctl != NULL, NULL );
 
+       node = ctl->listCriteria;
        cnt = g_list_length( ctl->listCriteria );
        ptrArray = g_new0( char *, 1 + cnt );
        i = 0;
-       node = ctl->listCriteria;
        while( node ) {
                ptrArray[ i++ ] = node->data;
+               /*debug_print("adding search attribute: %s\n", (gchar *) node->data);*/
+               node = g_list_next( node );
+       }
+       ptrArray[ i ] = NULL;
+       return ptrArray;
+}
+
+/**
+ * Return array of pointers to attributes for LDAP query.
+ * \param  ctl  Control object to process.
+ * \return NULL terminated list.
+ */
+char **ldapctl_full_attribute_array( LdapControl *ctl ) {
+       char **ptrArray;
+       GList *node, *def;
+       GList *tmp = NULL;
+       gint cnt, i;
+       cm_return_val_if_fail( ctl != NULL, NULL );
+
+       def = ctl->listCriteria;
+       while (def) {
+               tmp = g_list_append(tmp, g_strdup(def->data));
+               def = def->next;
+       }
+
+       def = ldapctl_get_default_criteria_list();
+       node = def;
+       
+       while (node) {
+               if( g_list_find_custom(tmp, (gpointer)def->data, 
+                               (GCompareFunc)strcmp2) == NULL) {
+                       tmp = g_list_append(tmp, g_strdup(node->data));
+               }
+               node = node->next;
+       }
+
+       g_list_free_full(def, g_free);
+
+       node = tmp;
+       cnt = g_list_length( tmp );
+       ptrArray = g_new0( char *, 1 + cnt);
+       i = 0;
+       while( node ) {
+               ptrArray[ i++ ] = node->data;
+               /*debug_print("adding search attribute: %s\n", (gchar *) node->data);*/
                node = g_list_next( node );
        }
        ptrArray[ i ] = NULL;
@@ -616,6 +663,7 @@ void ldapctl_free_attribute_array( char **ptrArray ) {
 
        /* Clear array to NULL's */
        for( i = 0; ptrArray[i] != NULL; i++ ) {
+               g_free(ptrArray[i]);
                ptrArray[i] = NULL;
        }
        g_free( ptrArray );
@@ -640,7 +688,7 @@ void ldapctl_parse_ldap_search( LdapControl *ctl, gchar *criteria ) {
        gchar *attrib;
        gint iLen;
 
-       g_return_if_fail( ctl != NULL );
+       cm_return_if_fail( ctl != NULL );
 
        ldapctl_criteria_list_clear( ctl );
        if( criteria == NULL ) return;
@@ -665,6 +713,75 @@ void ldapctl_parse_ldap_search( LdapControl *ctl, gchar *criteria ) {
        }
 }
 
+/**
+ * Return the default LDAP search criteria string.
+ * \return Formatted string or <i>""</i>. Should be g_free() when done.
+ */
+gchar *ldapctl_get_default_criteria() {
+       gchar *retVal = g_strdup(LDAPCTL_DFL_ATTR_LIST);
+       const gchar **attrs = ATTRIBUTE; 
+
+       while (*attrs) {
+               gchar *tmp = g_strdup_printf("%s, %s", retVal, *attrs++);
+               g_free(retVal);
+               retVal = tmp;
+       }
+       debug_print("default search criteria: %s\n", retVal);
+       return retVal;
+}
+
+/**
+ * Return the default LDAP search criteria list.
+ * \return GList or <i>NULL</i>.
+ */
+GList *ldapctl_get_default_criteria_list() {
+       gchar *criteria, *item;
+       gchar **c_list, **w_list;
+       GList *attr_list = NULL;
+
+       criteria = ldapctl_get_default_criteria();
+       c_list = g_strsplit(criteria, " ", 0);
+       g_free(criteria);
+       criteria = NULL;
+       w_list = c_list;
+       while ((criteria = *w_list++) != 0) {
+               /* copy string elimination <,> */
+               if (*w_list)
+                       item = g_strndup(criteria, strlen(criteria) - 1);
+               else
+                       item = g_strdup(criteria);
+               debug_print("adding attribute to list: %s\n", item);
+               attr_list = g_list_append(attr_list, item);
+       }
+       g_strfreev(c_list);
+       return attr_list;
+}
+
+/**
+ * Compare to GList for equality.
+ * \param l1 First GList
+ * \param l2 Second GList
+ * \Return TRUE or FALSE
+ */
+gboolean ldapctl_compare_list(GList *l1, GList *l2) {
+       gchar *first, *second;
+       if (! l1 && ! l2)
+               return TRUE;
+       if ((! l1 && l2) || (l1 && ! l2))
+               return FALSE;
+       while (l1 && l2) {
+               first = (gchar *) l1->data;
+               second = (gchar *) l2->data;
+               /*debug_print("comparing: %s = %s\n", first, second);*/
+               if ( ! (first && second) || strcmp(first, second) != 0) {
+                       return FALSE;
+               }
+               l1 = g_list_next(l1);
+               l2 = g_list_next(l2);
+       }
+       return TRUE;
+}
+
 #endif /* USE_LDAP */
 
 /*