void function cannot return a value (besides the function called was
[tprouty/samba.git] / source / passdb / pdb_ldap.c
index 0a160714198cb7d0663f05008f7d5f3e84d1b07d..ca7b1c8170bd12e8fa07e778085a4d6e190eee39 100644 (file)
@@ -1,11 +1,12 @@
 /* 
    Unix SMB/CIFS implementation.
    LDAP protocol helper functions for SAMBA
-   Copyright (C) Gerald Carter 2001
-   Copyright (C) Shahms King 2001
-   Copyright (C) Jean François Micouleau 1998
-   Copyright (C) Andrew Bartlett 2002
-   
+   Copyright (C) Jean François Micouleau       1998
+   Copyright (C) Gerald Carter                 2001
+   Copyright (C) Shahms King                   2001
+   Copyright (C) Andrew Bartlett               2002
+   Copyright (C) Stefan (metze) Metzmacher     2002
+    
    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
@@ -63,6 +64,7 @@ struct ldapsam_privates {
        LDAPMessage *entry;
        int index;
        
+       time_t last_ping;
        /* retrive-once info */
        const char *uri;
        
@@ -70,8 +72,15 @@ struct ldapsam_privates {
        
        uint32 low_nua_rid; 
        uint32 high_nua_rid; 
+
+       char *bind_dn;
+       char *bind_secret;
 };
 
+#define LDAPSAM_DONT_PING_TIME 10      /* ping only all 10 seconds */
+
+static struct ldapsam_privates *static_ldap_state;
+
 static uint32 ldapsam_get_next_available_nua_rid(struct ldapsam_privates *ldap_state);
 
 /*******************************************************************
@@ -136,44 +145,42 @@ static BOOL fetch_ldapsam_pw(char **dn, char** pw)
        return True;
 }
 
-char *attr[] = {"uid", "pwdLastSet", "logonTime",
-               "logoffTime", "kickoffTime", "cn",
-               "pwdCanChange", "pwdMustChange",
-               "dislplayName", "homeDrive",
-               "smbHome", "scriptPath",
-               "profilePath", "description",
-               "userWorkstation", "rid",
-               "primaryGroupID", "lmPassword",
-               "ntPassword", "acctFlags",
-               "domain", "description", NULL };
+static const char *attr[] = {"uid", "pwdLastSet", "logonTime",
+                            "logoffTime", "kickoffTime", "cn",
+                            "pwdCanChange", "pwdMustChange",
+                            "displayName", "homeDrive",
+                            "smbHome", "scriptPath",
+                            "profilePath", "description",
+                            "userWorkstations", "rid",
+                            "primaryGroupID", "lmPassword",
+                            "ntPassword", "acctFlags",
+                            "domain", "objectClass", 
+                            "uidNumber", "gidNumber", 
+                            "homeDirectory", NULL };
 
 /*******************************************************************
  open a connection to the ldap server.
 ******************************************************************/
-static BOOL ldapsam_open_connection (struct ldapsam_privates *ldap_state, LDAP ** ldap_struct)
+static int ldapsam_open_connection (struct ldapsam_privates *ldap_state, LDAP ** ldap_struct)
 {
+       int rc = LDAP_SUCCESS;
+       int version;
+       BOOL ldap_v3 = False;
 
-       if (geteuid() != 0) {
-               DEBUG(0, ("ldap_open_connection: cannot access LDAP when not root..\n"));
-               return False;
-       }
-       
 #if defined(LDAP_API_FEATURE_X_OPENLDAP) && (LDAP_API_VERSION > 2000)
        DEBUG(10, ("ldapsam_open_connection: %s\n", ldap_state->uri));
        
-       if (ldap_initialize(ldap_struct, ldap_state->uri) != LDAP_SUCCESS) {
-               DEBUG(0, ("ldap_initialize: %s\n", strerror(errno)));
-               return (False);
+       if ((rc = ldap_initialize(ldap_struct, ldap_state->uri)) != LDAP_SUCCESS) {
+               DEBUG(0, ("ldap_initialize: %s\n", ldap_err2string(rc)));
+               return rc;
        }
+       
 #else 
 
        /* Parse the string manually */
 
        {
-               int rc;
-               int tls = LDAP_OPT_X_TLS_HARD;
                int port = 0;
-               int version;
                fstring protocol;
                fstring host;
                const char *p = ldap_state->uri; 
@@ -183,7 +190,7 @@ static BOOL ldapsam_open_connection (struct ldapsam_privates *ldap_state, LDAP *
                if ( strncasecmp( p, "URL:", 4 ) == 0 ) {
                        p += 4;
                }
-
+               
                sscanf(p, "%10[^:]://%254s[^:]:%d", protocol, host, &port);
                
                if (port == 0) {
@@ -195,149 +202,452 @@ static BOOL ldapsam_open_connection (struct ldapsam_privates *ldap_state, LDAP *
                                DEBUG(0, ("unrecognised protocol (%s)!\n", protocol));
                        }
                }
-
+               
                if ((*ldap_struct = ldap_init(host, port)) == NULL)     {
                        DEBUG(0, ("ldap_init failed !\n"));
-                       return False;
+                       return LDAP_OPERATIONS_ERROR;
                }
-
-               /* Connect to older servers using SSL and V2 rather than Start TLS */
-               if (ldap_get_option(*ldap_struct, LDAP_OPT_PROTOCOL_VERSION, &version) == LDAP_OPT_SUCCESS)
-               {
-                       if (version != LDAP_VERSION2)
+               
+               if (strequal(protocol, "ldaps")) {
+#ifdef LDAP_OPT_X_TLS
+                       int tls = LDAP_OPT_X_TLS_HARD;
+                       if (ldap_set_option (*ldap_struct, LDAP_OPT_X_TLS, &tls) != LDAP_SUCCESS)
                        {
-                               version = LDAP_VERSION2;
-                               ldap_set_option (*ldap_struct, LDAP_OPT_PROTOCOL_VERSION, &version);
+                               DEBUG(0, ("Failed to setup a TLS session\n"));
                        }
+                       
+                       DEBUG(3,("LDAPS option set...!\n"));
+#else
+                       DEBUG(0,("ldapsam_open_connection: Secure connection not supported by LDAP client libraries!\n"));
+                       return LDAP_OPERATIONS_ERROR;
+#endif
                }
+       }
+#endif
 
-               if (strequal(protocol, "ldaps")) { 
-                       if (lp_ldap_ssl() == LDAP_SSL_START_TLS) {
-                               if (ldap_get_option (*ldap_struct, LDAP_OPT_PROTOCOL_VERSION, 
-                                                    &version) == LDAP_OPT_SUCCESS)
-                               {
-                                       if (version < LDAP_VERSION3)
-                                       {
-                                               version = LDAP_VERSION3;
-                                               ldap_set_option (*ldap_struct, LDAP_OPT_PROTOCOL_VERSION,
-                                                                &version);
-                                       }
-                               }
-                               if ((rc = ldap_start_tls_s (*ldap_struct, NULL, NULL)) != LDAP_SUCCESS)
-                               {
-                                       DEBUG(0,("Failed to issue the StartTLS instruction: %s\n",
-                                                ldap_err2string(rc)));
-                                       return False;
-                               }
-                               DEBUG (2, ("StartTLS issued: using a TLS connection\n"));
-                       } else {
-                               
-                               if (ldap_set_option (*ldap_struct, LDAP_OPT_X_TLS, &tls) != LDAP_SUCCESS)
-                               {
-                                       DEBUG(0, ("Failed to setup a TLS session\n"));
-                               }
+       if (ldap_get_option(*ldap_struct, LDAP_OPT_PROTOCOL_VERSION, &version) == LDAP_OPT_SUCCESS)
+       {
+               if (version != LDAP_VERSION3)
+               {
+                       version = LDAP_VERSION3;
+                       if (ldap_set_option (*ldap_struct, LDAP_OPT_PROTOCOL_VERSION, &version) == LDAP_OPT_SUCCESS) {
+                               ldap_v3 = True;
                        }
                } else {
-                       /* 
-                        * No special needs to setup options prior to the LDAP
-                        * bind (which should be called next via ldap_connect_system()
-                        */
+                       ldap_v3 = True;
                }
        }
+
+       if (lp_ldap_ssl() == LDAP_SSL_START_TLS) {
+#ifdef LDAP_OPT_X_TLS
+               if (ldap_v3) {
+                       if ((rc = ldap_start_tls_s (*ldap_struct, NULL, NULL)) != LDAP_SUCCESS)
+                       {
+                               DEBUG(0,("Failed to issue the StartTLS instruction: %s\n",
+                                        ldap_err2string(rc)));
+                               return rc;
+                       }
+                       DEBUG (3, ("StartTLS issued: using a TLS connection\n"));
+               } else {
+                       
+                       DEBUG(0, ("Need LDAPv3 for Start TLS\n"));
+                       return LDAP_OPERATIONS_ERROR;
+               }
+#else
+               DEBUG(0,("ldapsam_open_connection: StartTLS not supported by LDAP client libraries!\n"));
+               return LDAP_OPERATIONS_ERROR;
 #endif
+       }
 
-       DEBUG(2, ("ldap_open_connection: connection opened\n"));
-       return True;
+       DEBUG(2, ("ldapsam_open_connection: connection opened\n"));
+       return rc;
 }
 
 
 /*******************************************************************
- Add a rebind function for authenticated referrals
+ a rebind function for authenticated referrals
+ This version takes a void* that we can shove useful stuff in :-)
 ******************************************************************/
+#if defined(LDAP_API_FEATURE_X_OPENLDAP) && (LDAP_API_VERSION > 2000)
+#else
+static int rebindproc_with_state  (LDAP * ld, char **whop, char **credp, 
+                                  int *methodp, int freeit, void *arg)
+{
+       struct ldapsam_privates *ldap_state = arg;
+       
+       /** @TODO Should we be doing something to check what servers we rebind to?
+           Could we get a referral to a machine that we don't want to give our
+           username and password to? */
+       
+       if (freeit) {
+               SAFE_FREE(*whop);
+               memset(*credp, '\0', strlen(*credp));
+               SAFE_FREE(*credp);
+       } else {
+               DEBUG(5,("rebind_proc_with_state: Rebinding as \"%s\"\n", 
+                         ldap_state->bind_dn));
 
-static int rebindproc (LDAP *ldap_struct, char **whop, char **credp,
-                      int *method, int freeit )
+               *whop = strdup(ldap_state->bind_dn);
+               if (!*whop) {
+                       return LDAP_NO_MEMORY;
+               }
+               *credp = strdup(ldap_state->bind_secret);
+               if (!*credp) {
+                       SAFE_FREE(*whop);
+                       return LDAP_NO_MEMORY;
+               }
+               *methodp = LDAP_AUTH_SIMPLE;
+       }
+       return 0;
+}
+#endif /*defined(LDAP_API_FEATURE_X_OPENLDAP) && (LDAP_API_VERSION > 2000)*/
+
+/*******************************************************************
+ a rebind function for authenticated referrals
+ This version takes a void* that we can shove useful stuff in :-)
+ and actually does the connection.
+******************************************************************/
+#if defined(LDAP_API_FEATURE_X_OPENLDAP) && (LDAP_API_VERSION > 2000)
+static int rebindproc_connect_with_state (LDAP *ldap_struct, 
+                                         LDAP_CONST char *url, 
+                                         ber_tag_t request,
+                                         ber_int_t msgid, void *arg)
 {
+       struct ldapsam_privates *ldap_state = arg;
        int rc;
-       char *ldap_dn;
-       char *ldap_secret;
+       DEBUG(5,("rebindproc_connect_with_state: Rebinding as \"%s\"\n", 
+                ldap_state->bind_dn));
        
        /** @TODO Should we be doing something to check what servers we rebind to?
            Could we get a referral to a machine that we don't want to give our
            username and password to? */
+
+       rc = ldap_simple_bind_s(ldap_struct, ldap_state->bind_dn, ldap_state->bind_secret);
        
-       if (freeit != 0)
-       {
-               
-               if (!fetch_ldapsam_pw(&ldap_dn, &ldap_secret)) 
-               {
-                       DEBUG(0, ("ldap_connect_system: Failed to retrieve password from secrets.tdb\n"));
-                       return LDAP_OPERATIONS_ERROR;  /* No idea what to return */
-               }
-               
-               DEBUG(5,("ldap_connect_system: Rebinding as \"%s\"\n", 
-                         ldap_dn));
+       return rc;
+}
+#endif /*defined(LDAP_API_FEATURE_X_OPENLDAP) && (LDAP_API_VERSION > 2000)*/
 
-               rc = ldap_simple_bind_s(ldap_struct, ldap_dn, ldap_secret);
-               
-               SAFE_FREE(ldap_dn);
-               SAFE_FREE(ldap_secret);
+/*******************************************************************
+ Add a rebind function for authenticated referrals
+******************************************************************/
+#if defined(LDAP_API_FEATURE_X_OPENLDAP) && (LDAP_API_VERSION > 2000)
+#else
+# if LDAP_SET_REBIND_PROC_ARGS == 2
+static int rebindproc (LDAP *ldap_struct, char **whop, char **credp,
+                      int *method, int freeit )
+{
+       return rebindproc_with_state(ldap_struct, whop, credp,
+                                  method, freeit, static_ldap_state);
+       
+}
+# endif /*LDAP_SET_REBIND_PROC_ARGS == 2*/
+#endif /*defined(LDAP_API_FEATURE_X_OPENLDAP) && (LDAP_API_VERSION > 2000)*/
 
-               return rc;
-       }
-       return 0;
+/*******************************************************************
+ a rebind function for authenticated referrals
+ this also does the connection, but no void*.
+******************************************************************/
+#if defined(LDAP_API_FEATURE_X_OPENLDAP) && (LDAP_API_VERSION > 2000)
+# if LDAP_SET_REBIND_PROC_ARGS == 2
+static int rebindproc_connect (LDAP * ld, LDAP_CONST char *url, int request,
+                              ber_int_t msgid)
+{
+       return rebindproc_connect_with_state(ld, url, (ber_tag_t)request, msgid, 
+                                            static_ldap_state);
 }
+# endif /*LDAP_SET_REBIND_PROC_ARGS == 2*/
+#endif /*defined(LDAP_API_FEATURE_X_OPENLDAP) && (LDAP_API_VERSION > 2000)*/
 
 /*******************************************************************
  connect to the ldap server under system privilege.
 ******************************************************************/
-static BOOL ldapsam_connect_system(struct ldapsam_privates *ldap_state, LDAP * ldap_struct)
+static int ldapsam_connect_system(struct ldapsam_privates *ldap_state, LDAP * ldap_struct)
 {
        int rc;
        char *ldap_dn;
        char *ldap_secret;
 
+       /* The rebind proc needs this *HACK*.  We are not multithreaded, so
+          this will work, but it's not nice. */
+       static_ldap_state = ldap_state;
+
        /* get the password */
        if (!fetch_ldapsam_pw(&ldap_dn, &ldap_secret))
        {
                DEBUG(0, ("ldap_connect_system: Failed to retrieve password from secrets.tdb\n"));
-               return False;
+               return LDAP_INVALID_CREDENTIALS;
        }
 
+       ldap_state->bind_dn = ldap_dn;
+       ldap_state->bind_secret = ldap_secret;
+
        /* removed the sasl_bind_s "EXTERNAL" stuff, as my testsuite 
           (OpenLDAP) doesnt' seem to support it */
           
-       DEBUG(10,("ldap_connect_system: Binding to ldap server as \"%s\"\n",
-               ldap_dn));
-       
-       ldap_set_rebind_proc(ldap_struct, (LDAP_REBIND_PROC *)(&rebindproc));   
+       DEBUG(10,("ldap_connect_system: Binding to ldap server %s as \"%s\"\n",
+                 ldap_state->uri, ldap_dn));
 
-       rc = ldap_simple_bind_s(ldap_struct, ldap_dn, ldap_secret);
+#if defined(LDAP_API_FEATURE_X_OPENLDAP) && (LDAP_API_VERSION > 2000)
+# if LDAP_SET_REBIND_PROC_ARGS == 2    
+       ldap_set_rebind_proc(ldap_struct, &rebindproc_connect); 
+# endif
+# if LDAP_SET_REBIND_PROC_ARGS == 3    
+       ldap_set_rebind_proc(ldap_struct, &rebindproc_connect_with_state, (void *)ldap_state);  
+# endif
+#else /*defined(LDAP_API_FEATURE_X_OPENLDAP) && (LDAP_API_VERSION > 2000)*/
+# if LDAP_SET_REBIND_PROC_ARGS == 2    
+       ldap_set_rebind_proc(ldap_struct, &rebindproc); 
+# endif
+# if LDAP_SET_REBIND_PROC_ARGS == 3    
+       ldap_set_rebind_proc(ldap_struct, &rebindproc_with_state, (void *)ldap_state);  
+# endif
+#endif /*defined(LDAP_API_FEATURE_X_OPENLDAP) && (LDAP_API_VERSION > 2000)*/
 
-       SAFE_FREE(ldap_dn);
-       SAFE_FREE(ldap_secret);
+       rc = ldap_simple_bind_s(ldap_struct, ldap_dn, ldap_secret);
 
-       if (rc != LDAP_SUCCESS)
-       {
-               DEBUG(0, ("Bind failed: %s\n", ldap_err2string(rc)));
-               return False;
+       if (rc != LDAP_SUCCESS) {
+               char *ld_error;
+               ldap_get_option(ldap_state->ldap_struct, LDAP_OPT_ERROR_STRING,
+                               &ld_error);
+               DEBUG(0,
+                     ("failed to bind to server with dn= %s Error: %s\n\t%s\n",
+                              ldap_dn, ldap_err2string(rc),
+                              ld_error));
+               free(ld_error);
+               return rc;
        }
        
        DEBUG(2, ("ldap_connect_system: succesful connection to the LDAP server\n"));
-       return True;
+       return rc;
+}
+
+/**********************************************************************
+Connect to LDAP server 
+*********************************************************************/
+static int ldapsam_open(struct ldapsam_privates *ldap_state)
+{
+       int rc;
+       SMB_ASSERT(ldap_state);
+               
+#ifndef NO_LDAP_SECURITY
+       if (geteuid() != 0) {
+               DEBUG(0, ("ldapsam_open: cannot access LDAP when not root..\n"));
+               return  LDAP_INSUFFICIENT_ACCESS;
+       }
+#endif
+
+       if ((ldap_state->ldap_struct != NULL) && ((ldap_state->last_ping + LDAPSAM_DONT_PING_TIME) < time(NULL))) {
+               struct sockaddr_un addr;
+               socklen_t len;
+               int sd;
+               if (ldap_get_option(ldap_state->ldap_struct, LDAP_OPT_DESC, &sd) == 0 &&
+                   getpeername(sd, (struct sockaddr *) &addr, &len) < 0) {
+                       /* the other end has died. reopen. */
+                       ldap_unbind_ext(ldap_state->ldap_struct, NULL, NULL);
+                       ldap_state->ldap_struct = NULL;
+                       ldap_state->last_ping = (time_t)0;
+               } else {
+                       ldap_state->last_ping = time(NULL);
+               } 
+       }
+
+       if (ldap_state->ldap_struct != NULL) {
+               DEBUG(5,("ldapsam_open: allready connected to the LDAP server\n"));
+               return LDAP_SUCCESS;
+       }
+
+       if ((rc = ldapsam_open_connection(ldap_state, &ldap_state->ldap_struct))) {
+               return rc;
+       }
+
+       if ((rc = ldapsam_connect_system(ldap_state, ldap_state->ldap_struct))) {
+               ldap_unbind_ext(ldap_state->ldap_struct, NULL, NULL);
+               ldap_state->ldap_struct = NULL;
+               return rc;
+       }
+
+
+       ldap_state->last_ping = time(NULL);
+       DEBUG(4,("The LDAP server is succesful connected\n"));
+
+       return LDAP_SUCCESS;
+}
+
+/**********************************************************************
+Disconnect from LDAP server 
+*********************************************************************/
+static NTSTATUS ldapsam_close(struct ldapsam_privates *ldap_state)
+{
+       if (!ldap_state)
+               return NT_STATUS_INVALID_PARAMETER;
+               
+       if (ldap_state->ldap_struct != NULL) {
+               ldap_unbind_ext(ldap_state->ldap_struct, NULL, NULL);
+               ldap_state->ldap_struct = NULL;
+       }
+       
+       DEBUG(5,("The connection to the LDAP server was closed\n"));
+       /* maybe free the results here --metze */
+       
+       return NT_STATUS_OK;
+}
+
+static int ldapsam_retry_open(struct ldapsam_privates *ldap_state, int *attempts)
+{
+       int rc;
+
+       SMB_ASSERT(ldap_state && attempts);
+               
+       if (*attempts != 0) {
+               /* we retry after 0.5, 2, 4.5, 8, 12.5, 18, 24.5 seconds */
+               msleep((((*attempts)*(*attempts))/2)*1000);
+       }
+       (*attempts)++;
+
+       if ((rc = ldapsam_open(ldap_state))) {
+               DEBUG(0,("Connection to LDAP Server failed for the %d try!\n",*attempts));
+               return rc;
+       } 
+       
+       return LDAP_SUCCESS;            
+}
+
+
+static int ldapsam_search(struct ldapsam_privates *ldap_state, 
+                         const char *base, int scope, const char *filter, 
+                         const char *attrs[], int attrsonly, 
+                         LDAPMessage **res)
+{
+       int             rc = LDAP_SERVER_DOWN;
+       int             attempts = 0;
+       
+       SMB_ASSERT(ldap_state);
+
+       while ((rc == LDAP_SERVER_DOWN) && (attempts < 8)) {
+               
+               if ((rc = ldapsam_retry_open(ldap_state,&attempts)) != LDAP_SUCCESS)
+                       continue;
+               
+               rc = ldap_search_s(ldap_state->ldap_struct, base, scope, 
+                                  filter, attrs, attrsonly, res);
+       }
+       
+       if (rc == LDAP_SERVER_DOWN) {
+               DEBUG(0,("%s: LDAP server is down!\n",FUNCTION_MACRO));
+               ldapsam_close(ldap_state);      
+       }
+       
+       return rc;
+}
+
+static int ldapsam_modify(struct ldapsam_privates *ldap_state, char *dn, LDAPMod *attrs[])
+{
+       int             rc = LDAP_SERVER_DOWN;
+       int             attempts = 0;
+       
+       if (!ldap_state)
+               return (-1);
+
+       while ((rc == LDAP_SERVER_DOWN) && (attempts < 8)) {
+               
+               if ((rc = ldapsam_retry_open(ldap_state,&attempts)) != LDAP_SUCCESS)
+                       continue;
+               
+               rc = ldap_modify_s(ldap_state->ldap_struct, dn, attrs);
+       }
+       
+       if (rc == LDAP_SERVER_DOWN) {
+               DEBUG(0,("%s: LDAP server is down!\n",FUNCTION_MACRO));
+               ldapsam_close(ldap_state);      
+       }
+       
+       return rc;
+}
+
+static int ldapsam_add(struct ldapsam_privates *ldap_state, const char *dn, LDAPMod *attrs[])
+{
+       int             rc = LDAP_SERVER_DOWN;
+       int             attempts = 0;
+       
+       if (!ldap_state)
+               return (-1);
+
+       while ((rc == LDAP_SERVER_DOWN) && (attempts < 8)) {
+               
+               if ((rc = ldapsam_retry_open(ldap_state,&attempts)) != LDAP_SUCCESS)
+                       continue;
+               
+               rc = ldap_add_s(ldap_state->ldap_struct, dn, attrs);
+       }
+       
+       if (rc == LDAP_SERVER_DOWN) {
+               DEBUG(0,("%s: LDAP server is down!\n",FUNCTION_MACRO));
+               ldapsam_close(ldap_state);      
+       }
+               
+       return rc;
+}
+
+static int ldapsam_delete(struct ldapsam_privates *ldap_state, char *dn)
+{
+       int             rc = LDAP_SERVER_DOWN;
+       int             attempts = 0;
+       
+       if (!ldap_state)
+               return (-1);
+
+       while ((rc == LDAP_SERVER_DOWN) && (attempts < 8)) {
+               
+               if ((rc = ldapsam_retry_open(ldap_state,&attempts)) != LDAP_SUCCESS)
+                       continue;
+               
+               rc = ldap_delete_s(ldap_state->ldap_struct, dn);
+       }
+       
+       if (rc == LDAP_SERVER_DOWN) {
+               DEBUG(0,("%s: LDAP server is down!\n",FUNCTION_MACRO));
+               ldapsam_close(ldap_state);      
+       }
+               
+       return rc;
+}
+
+static int ldapsam_extended_operation(struct ldapsam_privates *ldap_state, LDAP_CONST char *reqoid, struct berval *reqdata, LDAPControl **serverctrls, LDAPControl **clientctrls, char **retoidp, struct berval **retdatap)
+{
+       int             rc = LDAP_SERVER_DOWN;
+       int             attempts = 0;
+       
+       if (!ldap_state)
+               return (-1);
+
+       while ((rc == LDAP_SERVER_DOWN) && (attempts < 8)) {
+               
+               if ((rc = ldapsam_retry_open(ldap_state,&attempts)) != LDAP_SUCCESS)
+                       continue;
+               
+               rc = ldap_extended_operation_s(ldap_state->ldap_struct, reqoid, reqdata, serverctrls, clientctrls, retoidp, retdatap);
+       }
+       
+       if (rc == LDAP_SERVER_DOWN) {
+               DEBUG(0,("%s: LDAP server is down!\n",FUNCTION_MACRO));
+               ldapsam_close(ldap_state);      
+       }
+               
+       return rc;
 }
 
 /*******************************************************************
  run the search by name.
 ******************************************************************/
-static int ldapsam_search_one_user (struct ldapsam_privates *ldap_state, LDAP * ldap_struct, const char *filter, LDAPMessage ** result)
+static int ldapsam_search_one_user (struct ldapsam_privates *ldap_state, const char *filter, LDAPMessage ** result)
 {
        int scope = LDAP_SCOPE_SUBTREE;
        int rc;
 
        DEBUG(2, ("ldapsam_search_one_user: searching for:[%s]\n", filter));
 
-       rc = ldap_search_s(ldap_struct, lp_ldap_suffix (), scope, filter, attr, 0, result);
+       rc = ldapsam_search(ldap_state, lp_ldap_suffix (), scope, filter, attr, 0, result);
 
        if (rc != LDAP_SUCCESS) {
                DEBUG(0,("ldapsam_search_one_user: Problem during the LDAP search: %s\n", 
@@ -352,11 +662,16 @@ static int ldapsam_search_one_user (struct ldapsam_privates *ldap_state, LDAP *
 /*******************************************************************
  run the search by name.
 ******************************************************************/
-static int ldapsam_search_one_user_by_name (struct ldapsam_privates *ldap_state, LDAP * ldap_struct, const char *user,
+static int ldapsam_search_one_user_by_name (struct ldapsam_privates *ldap_state, const char *user,
                             LDAPMessage ** result)
 {
        pstring filter;
-       
+       char *escape_user = escape_ldap_string_alloc(user);
+
+       if (!escape_user) {
+               return LDAP_NO_MEMORY;
+       }
+
        /*
         * in the filter expression, replace %u with the real name
         * so in ldap filter, %u MUST exist :-)
@@ -367,20 +682,24 @@ static int ldapsam_search_one_user_by_name (struct ldapsam_privates *ldap_state,
         * have to use this here because $ is filtered out
           * in pstring_sub
         */
-       all_string_sub(filter, "%u", user, sizeof(pstring));
+       
 
-       return ldapsam_search_one_user(ldap_state, ldap_struct, filter, result);
+       all_string_sub(filter, "%u", escape_user, sizeof(pstring));
+       SAFE_FREE(escape_user);
+
+       return ldapsam_search_one_user(ldap_state, filter, result);
 }
 
 /*******************************************************************
  run the search by uid.
 ******************************************************************/
 static int ldapsam_search_one_user_by_uid(struct ldapsam_privates *ldap_state, 
-                                         LDAP * ldap_struct, int uid,
+                                         int uid,
                                          LDAPMessage ** result)
 {
        struct passwd *user;
        pstring filter;
+       char *escape_user;
 
        /* Get the username from the system and look that up in the LDAP */
        
@@ -391,18 +710,25 @@ static int ldapsam_search_one_user_by_uid(struct ldapsam_privates *ldap_state,
        
        pstrcpy(filter, lp_ldap_filter());
        
-       all_string_sub(filter, "%u", user->pw_name, sizeof(pstring));
+       escape_user = escape_ldap_string_alloc(user->pw_name);
+       if (!escape_user) {
+               passwd_free(&user);
+               return LDAP_NO_MEMORY;
+       }
+
+       all_string_sub(filter, "%u", escape_user, sizeof(pstring));
 
        passwd_free(&user);
+       SAFE_FREE(escape_user);
 
-       return ldapsam_search_one_user(ldap_state, ldap_struct, filter, result);
+       return ldapsam_search_one_user(ldap_state, filter, result);
 }
 
 /*******************************************************************
  run the search by rid.
 ******************************************************************/
 static int ldapsam_search_one_user_by_rid (struct ldapsam_privates *ldap_state, 
-                                          LDAP * ldap_struct, uint32 rid,
+                                          uint32 rid,
                                           LDAPMessage ** result)
 {
        pstring filter;
@@ -411,10 +737,10 @@ static int ldapsam_search_one_user_by_rid (struct ldapsam_privates *ldap_state,
        /* check if the user rid exsists, if not, try searching on the uid */
        
        snprintf(filter, sizeof(filter) - 1, "rid=%i", rid);
-       rc = ldapsam_search_one_user(ldap_state, ldap_struct, filter, result);
+       rc = ldapsam_search_one_user(ldap_state, filter, result);
        
        if (rc != LDAP_SUCCESS)
-               rc = ldapsam_search_one_user_by_uid(ldap_state, ldap_struct, 
+               rc = ldapsam_search_one_user_by_uid(ldap_state,
                                                    fallback_pdb_user_rid_to_uid(rid), 
                                                    result);
 
@@ -425,7 +751,7 @@ static int ldapsam_search_one_user_by_rid (struct ldapsam_privates *ldap_state,
 search an attribute and return the first value found.
 ******************************************************************/
 static BOOL get_single_attribute (LDAP * ldap_struct, LDAPMessage * entry,
-                                 char *attribute, pstring value)
+                                 const char *attribute, pstring value)
 {
        char **values;
 
@@ -460,8 +786,11 @@ static void make_a_mod (LDAPMod *** modlist, int modop, const char *attribute, c
        if (attribute == NULL || *attribute == '\0')
                return;
 
-       if (value == NULL || *value == '\0')
+#if 0
+       /* Why do we need this??? -- vl */
+               if (value == NULL || *value == '\0')
                return;
+#endif
 
        if (mods == NULL) 
        {
@@ -518,15 +847,147 @@ static void make_a_mod (LDAPMod *** modlist, int modop, const char *attribute, c
        *modlist = mods;
 }
 
+/*******************************************************************
+ Delete complete object or objectclass and attrs from
+ object found in search_result depending on lp_ldap_del_only_sam
+******************************************************************/
+static NTSTATUS ldapsam_delete_entry(struct ldapsam_privates *ldap_state,
+                                    LDAPMessage *result,
+                                    const char *objectclass,
+                                    const char **attrs)
+{
+       int rc;
+       LDAPMessage *entry;
+       LDAPMod **mods = NULL;
+       char *name, *dn;
+       BerElement *ptr = NULL;
+
+       rc = ldap_count_entries(ldap_state->ldap_struct, result);
+
+       if (rc != 1) {
+               DEBUG(0, ("Entry must exist exactly once!\n"));
+               return NT_STATUS_UNSUCCESSFUL;
+       }
+
+       entry = ldap_first_entry(ldap_state->ldap_struct, result);
+       dn    = ldap_get_dn(ldap_state->ldap_struct, entry);
+
+       if (!lp_ldap_del_only_sam()) {
+               NTSTATUS ret = NT_STATUS_OK;
+               rc = ldapsam_delete(ldap_state, dn);
+
+               if (rc != LDAP_SUCCESS) {
+                       DEBUG(0, ("Could not delete object %s\n", dn));
+                       ret = NT_STATUS_UNSUCCESSFUL;
+               }
+               ldap_memfree(dn);
+               return ret;
+       }
+
+       /* Ok, delete only the SAM attributes */
+
+       for (name = ldap_first_attribute(ldap_state->ldap_struct, entry, &ptr);
+            name != NULL;
+            name = ldap_next_attribute(ldap_state->ldap_struct, entry, ptr)) {
+
+               const char **attrib;
+
+               /* We are only allowed to delete the attributes that
+                  really exist. */
+
+               for (attrib = attrs; *attrib != NULL; attrib++) {
+                       if (StrCaseCmp(*attrib, name) == 0) {
+                               DEBUG(10, ("deleting attribute %s\n", name));
+                               make_a_mod(&mods, LDAP_MOD_DELETE, name, NULL);
+                       }
+               }
+
+               ldap_memfree(name);
+       }
+
+       if (ptr != NULL) {
+               ber_free(ptr, 0);
+       }
+       
+       make_a_mod(&mods, LDAP_MOD_DELETE, "objectClass", objectclass);
+
+       rc = ldapsam_modify(ldap_state, dn, mods);
+       ldap_mods_free(mods, 1);
+
+       if (rc != LDAP_SUCCESS) {
+               DEBUG(0, ("could not delete attributes for %s, error: %s\n",
+                         dn, ldap_err2string(rc)));
+               ldap_memfree(dn);
+               return NT_STATUS_UNSUCCESSFUL;
+       }
+
+       ldap_memfree(dn);
+       return NT_STATUS_OK;
+}
+                                         
 /* New Interface is being implemented here */
 
 /**********************************************************************
-Initialize SAM_ACCOUNT from an LDAP query
-(Based on init_sam_from_buffer in pdb_tdb.c)
+Initialize SAM_ACCOUNT from an LDAP query (unix attributes only)
 *********************************************************************/
-static BOOL init_sam_from_ldap (struct ldapsam_privates *ldap_state, 
+static BOOL get_unix_attributes (struct ldapsam_privates *ldap_state, 
                                SAM_ACCOUNT * sampass,
-                               LDAP * ldap_struct, LDAPMessage * entry)
+                               LDAPMessage * entry)
+{
+       pstring  homedir;
+       pstring  temp;
+       uid_t uid;
+       gid_t gid;
+       char **ldap_values;
+       char **values;
+
+       if ((ldap_values = ldap_get_values (ldap_state->ldap_struct, entry, "objectClass")) == NULL) {
+               DEBUG (1, ("get_unix_attributes: no objectClass! \n"));
+               return False;
+       }
+
+       for (values=ldap_values;*values;values++) {
+               if (strcasecmp(*values, "posixAccount") == 0) {
+                       break;
+               }
+       }
+       
+       if (!*values) { /*end of array, no posixAccount */
+               DEBUG(10, ("user does not have posixAcccount attributes\n"));
+               ldap_value_free(ldap_values);
+               return False;
+       }
+       ldap_value_free(ldap_values);
+
+       if (!get_single_attribute(ldap_state->ldap_struct, entry, "homeDirectory", homedir)) 
+               return False;
+       
+       if (!get_single_attribute(ldap_state->ldap_struct, entry, "uidNumber", temp))
+               return False;
+       
+       uid = (uid_t)atol(temp);
+       
+       if (!get_single_attribute(ldap_state->ldap_struct, entry, "gidNumber", temp))
+               return False;
+       
+       gid = (gid_t)atol(temp);
+
+       pdb_set_unix_homedir(sampass, homedir, PDB_SET);
+       pdb_set_uid(sampass, uid, PDB_SET);
+       pdb_set_gid(sampass, gid, PDB_SET);
+       
+       DEBUG(10, ("user has posixAcccount attributes\n"));
+       return True;
+}
+
+
+/**********************************************************************
+Initialize SAM_ACCOUNT from an LDAP query
+(Based on init_sam_from_buffer in pdb_tdb.c)
+*********************************************************************/
+static BOOL init_sam_from_ldap (struct ldapsam_privates *ldap_state, 
+                               SAM_ACCOUNT * sampass,
+                               LDAPMessage * entry)
 {
        time_t  logon_time,
                        logoff_time,
@@ -548,9 +1009,9 @@ static BOOL init_sam_from_ldap (struct ldapsam_privates *ldap_state,
        struct passwd   *pw;
        uint32          user_rid, 
                        group_rid;
-       uint8           smblmpwd[16],
-                       smbntpwd[16];
-       uint16          acct_ctrl, 
+       uint8           smblmpwd[LM_HASH_LEN],
+                       smbntpwd[NT_HASH_LEN];
+       uint16          acct_ctrl = 0
                        logon_divs;
        uint32 hours_len;
        uint8           hours[MAX_HOURS_LEN];
@@ -575,104 +1036,121 @@ static BOOL init_sam_from_ldap (struct ldapsam_privates *ldap_state,
        workstations[0] = '\0';
         
 
-       if (sampass == NULL || ldap_struct == NULL || entry == NULL) {
+       if (sampass == NULL || ldap_state == NULL || entry == NULL) {
                DEBUG(0, ("init_sam_from_ldap: NULL parameters found!\n"));
                return False;
        }
 
-       get_single_attribute(ldap_struct, entry, "uid", username);
+       if (ldap_state->ldap_struct == NULL) {
+               DEBUG(0, ("init_sam_from_ldap: ldap_state->ldap_struct is NULL!\n"));
+               return False;
+       }
+       
+       get_single_attribute(ldap_state->ldap_struct, entry, "uid", username);
        DEBUG(2, ("Entry found for user: %s\n", username));
 
        pstrcpy(nt_username, username);
 
        pstrcpy(domain, lp_workgroup());
+       
+       pdb_set_username(sampass, username, PDB_SET);
 
-       get_single_attribute(ldap_struct, entry, "rid", temp);
+       pdb_set_domain(sampass, domain, PDB_DEFAULT);
+       pdb_set_nt_username(sampass, nt_username, PDB_SET);
+
+       get_single_attribute(ldap_state->ldap_struct, entry, "rid", temp);
        user_rid = (uint32)atol(temp);
-       if (!get_single_attribute(ldap_struct, entry, "primaryGroupID", temp)) {
+
+       pdb_set_user_sid_from_rid(sampass, user_rid, PDB_SET);
+
+       if (!get_single_attribute(ldap_state->ldap_struct, entry, "primaryGroupID", temp)) {
                group_rid = 0;
        } else {
                group_rid = (uint32)atol(temp);
+               pdb_set_group_sid_from_rid(sampass, group_rid, PDB_SET);
        }
 
-       if ((ldap_state->permit_non_unix_accounts) 
-           && (user_rid >= ldap_state->low_nua_rid)
-           && (user_rid <= ldap_state->high_nua_rid)) {
-               
-       } else {
+
+       /* 
+        * If so configured, try and get the values from LDAP 
+        */
+
+       if (!lp_ldap_trust_ids() || (!get_unix_attributes(ldap_state, sampass, entry))) {
                
-               /* These values MAY be in LDAP, but they can also be retrieved through 
-                *  sys_getpw*() which is how we're doing it 
+               /* 
+                * Otherwise just ask the system getpw() calls.
                 */
        
                pw = getpwnam_alloc(username);
                if (pw == NULL) {
-                       DEBUG (2,("init_sam_from_ldap: User [%s] does not ave a uid!\n", username));
-                       return False;
+                       if (! ldap_state->permit_non_unix_accounts) {
+                               DEBUG (2,("init_sam_from_ldap: User [%s] does not exist via system getpwnam!\n", username));
+                               return False;
+                       }
+               } else {
+                       uid = pw->pw_uid;
+                       pdb_set_uid(sampass, uid, PDB_SET);
+                       gid = pw->pw_gid;
+                       pdb_set_gid(sampass, gid, PDB_SET);
+                       
+                       pdb_set_unix_homedir(sampass, pw->pw_dir, PDB_SET);
+
+                       passwd_free(&pw);
                }
-               uid = pw->pw_uid;
-               gid = pw->pw_gid;
-
-               pdb_set_unix_homedir(sampass, pw->pw_dir);
-
-               passwd_free(&pw);
-
-               pdb_set_uid(sampass, uid);
-               pdb_set_gid(sampass, gid);
+       }
 
-               if (group_rid == 0) {
-                       GROUP_MAP map;
-                       /* call the mapping code here */
-                       if(get_group_map_from_gid(gid, &map, MAPPING_WITHOUT_PRIV)) {
-                               if (!sid_peek_check_rid(get_global_sam_sid(), &map.sid, &group_rid))
-                                       return False;
-                       } 
-                       else {
-                               group_rid=pdb_gid_to_group_rid(gid);
-                       }
+       if (group_rid == 0 && pdb_get_init_flags(sampass,PDB_GID) != PDB_DEFAULT) {
+               GROUP_MAP map;
+               gid = pdb_get_gid(sampass);
+               /* call the mapping code here */
+               if(pdb_getgrgid(&map, gid, MAPPING_WITHOUT_PRIV)) {
+                       pdb_set_group_sid(sampass, &map.sid, PDB_SET);
+               } 
+               else {
+                       pdb_set_group_sid_from_rid(sampass, pdb_gid_to_group_rid(gid), PDB_SET);
                }
        }
 
-       if (!get_single_attribute(ldap_struct, entry, "pwdLastSet", temp)) {
+       if (!get_single_attribute(ldap_state->ldap_struct, entry, "pwdLastSet", temp)) {
                /* leave as default */
        } else {
                pass_last_set_time = (time_t) atol(temp);
-               pdb_set_pass_last_set_time(sampass, pass_last_set_time);
+               pdb_set_pass_last_set_time(sampass, pass_last_set_time, PDB_SET);
        }
 
-       if (!get_single_attribute(ldap_struct, entry, "logonTime", temp)) {
+       if (!get_single_attribute(ldap_state->ldap_struct, entry, "logonTime", temp)) {
                /* leave as default */
        } else {
                logon_time = (time_t) atol(temp);
-               pdb_set_logon_time(sampass, logon_time, True);
+               pdb_set_logon_time(sampass, logon_time, PDB_SET);
        }
 
-       if (!get_single_attribute(ldap_struct, entry, "logoffTime", temp)) {
+       if (!get_single_attribute(ldap_state->ldap_struct, entry, "logoffTime", temp)) {
                /* leave as default */
        } else {
                logoff_time = (time_t) atol(temp);
-               pdb_set_logoff_time(sampass, logoff_time, True);
+               pdb_set_logoff_time(sampass, logoff_time, PDB_SET);
        }
 
-       if (!get_single_attribute(ldap_struct, entry, "kickoffTime", temp)) {
+       if (!get_single_attribute(ldap_state->ldap_struct, entry, "kickoffTime", temp)) {
                /* leave as default */
        } else {
                kickoff_time = (time_t) atol(temp);
-               pdb_set_kickoff_time(sampass, kickoff_time, True);
+               pdb_set_kickoff_time(sampass, kickoff_time, PDB_SET);
        }
 
-       if (!get_single_attribute(ldap_struct, entry, "pwdCanChange", temp)) {
+       if (!get_single_attribute(ldap_state->ldap_struct, entry, "pwdCanChange", temp)) {
                /* leave as default */
        } else {
                pass_can_change_time = (time_t) atol(temp);
-               pdb_set_pass_can_change_time(sampass, pass_can_change_time, True);
+               pdb_set_pass_can_change_time(sampass, pass_can_change_time, PDB_SET);
        }
 
-       if (!get_single_attribute(ldap_struct, entry, "pwdMustChange", temp)) {
+       if (!get_single_attribute(ldap_state->ldap_struct, entry, "pwdMustChange", temp)) {
                /* leave as default */
        } else {
                pass_must_change_time = (time_t) atol(temp);
-               pdb_set_pass_must_change_time(sampass, pass_must_change_time, True);
+               pdb_set_pass_must_change_time(sampass, pass_must_change_time, PDB_SET);
        }
 
        /* recommend that 'gecos' and 'displayName' should refer to the same
@@ -681,66 +1159,66 @@ static BOOL init_sam_from_ldap (struct ldapsam_privates *ldap_state,
         * that fits your needs; using cn then displayName rather than 'userFullName'
         */
 
-       if (!get_single_attribute(ldap_struct, entry, "cn", fullname)) {
-               if (!get_single_attribute(ldap_struct, entry, "displayName", fullname)) {
+       if (!get_single_attribute(ldap_state->ldap_struct, entry, "cn", fullname)) {
+               if (!get_single_attribute(ldap_state->ldap_struct, entry, "displayName", fullname)) {
                        /* leave as default */
                } else {
-                       pdb_set_fullname(sampass, fullname);
+                       pdb_set_fullname(sampass, fullname, PDB_SET);
                }
        } else {
-               pdb_set_fullname(sampass, fullname);
+               pdb_set_fullname(sampass, fullname, PDB_SET);
        }
 
-       if (!get_single_attribute(ldap_struct, entry, "homeDrive", dir_drive)) {
-               pdb_set_dir_drive(sampass, standard_sub_specified(sampass->mem_ctx, 
-                                                                 lp_logon_path(),
+       if (!get_single_attribute(ldap_state->ldap_struct, entry, "homeDrive", dir_drive)) {
+               pdb_set_dir_drive(sampass, talloc_sub_specified(sampass->mem_ctx, 
+                                                                 lp_logon_drive(),
                                                                  username, domain, 
                                                                  uid, gid),
-                                 False);
+                                 PDB_DEFAULT);
        } else {
-               pdb_set_dir_drive(sampass, dir_drive, True);
+               pdb_set_dir_drive(sampass, dir_drive, PDB_SET);
        }
 
-       if (!get_single_attribute(ldap_struct, entry, "smbHome", homedir)) {
-               pdb_set_dir_drive(sampass, standard_sub_specified(sampass->mem_ctx, 
+       if (!get_single_attribute(ldap_state->ldap_struct, entry, "smbHome", homedir)) {
+               pdb_set_homedir(sampass, talloc_sub_specified(sampass->mem_ctx, 
                                                                  lp_logon_home(),
                                                                  username, domain, 
                                                                  uid, gid), 
-                                 False);
+                                 PDB_DEFAULT);
        } else {
-               pdb_set_homedir(sampass, homedir, True);
+               pdb_set_homedir(sampass, homedir, PDB_SET);
        }
 
-       if (!get_single_attribute(ldap_struct, entry, "scriptPath", logon_script)) {
-               pdb_set_logon_script(sampass, standard_sub_specified(sampass->mem_ctx, 
+       if (!get_single_attribute(ldap_state->ldap_struct, entry, "scriptPath", logon_script)) {
+               pdb_set_logon_script(sampass, talloc_sub_specified(sampass->mem_ctx, 
                                                                     lp_logon_script(),
                                                                     username, domain, 
                                                                     uid, gid), 
-                                    False);
+                                    PDB_DEFAULT);
        } else {
-               pdb_set_logon_script(sampass, logon_script, True);
+               pdb_set_logon_script(sampass, logon_script, PDB_SET);
        }
 
-       if (!get_single_attribute(ldap_struct, entry, "profilePath", profile_path)) {
-               pdb_set_profile_path(sampass, standard_sub_specified(sampass->mem_ctx, 
+       if (!get_single_attribute(ldap_state->ldap_struct, entry, "profilePath", profile_path)) {
+               pdb_set_profile_path(sampass, talloc_sub_specified(sampass->mem_ctx, 
                                                                     lp_logon_path(),
                                                                     username, domain, 
                                                                     uid, gid), 
-                                    False);
+                                    PDB_DEFAULT);
        } else {
-               pdb_set_profile_path(sampass, profile_path, True);
+               pdb_set_profile_path(sampass, profile_path, PDB_SET);
        }
 
-       if (!get_single_attribute(ldap_struct, entry, "description", acct_desc)) {
+       if (!get_single_attribute(ldap_state->ldap_struct, entry, "description", acct_desc)) {
                /* leave as default */
        } else {
-               pdb_set_acct_desc(sampass, acct_desc);
+               pdb_set_acct_desc(sampass, acct_desc, PDB_SET);
        }
 
-       if (!get_single_attribute(ldap_struct, entry, "userWorkstations", workstations)) {
+       if (!get_single_attribute(ldap_state->ldap_struct, entry, "userWorkstations", workstations)) {
                /* leave as default */;
        } else {
-               pdb_set_workstations(sampass, workstations);
+               pdb_set_workstations(sampass, workstations, PDB_SET);
        }
 
        /* FIXME: hours stuff should be cleaner */
@@ -749,25 +1227,27 @@ static BOOL init_sam_from_ldap (struct ldapsam_privates *ldap_state,
        hours_len = 21;
        memset(hours, 0xff, hours_len);
 
-       if (!get_single_attribute (ldap_struct, entry, "lmPassword", temp)) {
+       if (!get_single_attribute (ldap_state->ldap_struct, entry, "lmPassword", temp)) {
                /* leave as default */
        } else {
                pdb_gethexpwd(temp, smblmpwd);
-               memset((char *)temp, '\0', sizeof(temp));
-               if (!pdb_set_lanman_passwd(sampass, smblmpwd))
+               memset((char *)temp, '\0', strlen(temp)+1);
+               if (!pdb_set_lanman_passwd(sampass, smblmpwd, PDB_SET))
                        return False;
+               ZERO_STRUCT(smblmpwd);
        }
 
-       if (!get_single_attribute (ldap_struct, entry, "ntPassword", temp)) {
+       if (!get_single_attribute (ldap_state->ldap_struct, entry, "ntPassword", temp)) {
                /* leave as default */
        } else {
                pdb_gethexpwd(temp, smbntpwd);
-               memset((char *)temp, '\0', sizeof(temp));
-               if (!pdb_set_nt_passwd(sampass, smbntpwd))
+               memset((char *)temp, '\0', strlen(temp)+1);
+               if (!pdb_set_nt_passwd(sampass, smbntpwd, PDB_SET))
                        return False;
+               ZERO_STRUCT(smbntpwd);
        }
 
-       if (!get_single_attribute (ldap_struct, entry, "acctFlags", temp)) {
+       if (!get_single_attribute (ldap_state->ldap_struct, entry, "acctFlags", temp)) {
                acct_ctrl |= ACB_NORMAL;
        } else {
                acct_ctrl = pdb_decode_acct_ctrl(temp);
@@ -775,37 +1255,38 @@ static BOOL init_sam_from_ldap (struct ldapsam_privates *ldap_state,
                if (acct_ctrl == 0)
                        acct_ctrl |= ACB_NORMAL;
 
-               pdb_set_acct_ctrl(sampass, acct_ctrl);
+               pdb_set_acct_ctrl(sampass, acct_ctrl, PDB_SET);
        }
 
-       pdb_set_hours_len(sampass, hours_len);
-       pdb_set_logon_divs(sampass, logon_divs);
-
-       pdb_set_user_sid_from_rid(sampass, user_rid);
-       pdb_set_group_sid_from_rid(sampass, group_rid);
-
-       pdb_set_username(sampass, username);
-
-       pdb_set_domain(sampass, domain);
-       pdb_set_nt_username(sampass, nt_username);
+       pdb_set_hours_len(sampass, hours_len, PDB_SET);
+       pdb_set_logon_divs(sampass, logon_divs, PDB_SET);
 
-       pdb_set_munged_dial(sampass, munged_dial);
+       pdb_set_munged_dial(sampass, munged_dial, PDB_SET);
        
-       /* pdb_set_unknown_3(sampass, unknown3); */
-       /* pdb_set_unknown_5(sampass, unknown5); */
-       /* pdb_set_unknown_6(sampass, unknown6); */
+       /* pdb_set_unknown_3(sampass, unknown3, PDB_SET); */
+       /* pdb_set_unknown_5(sampass, unknown5, PDB_SET); */
+       /* pdb_set_unknown_6(sampass, unknown6, PDB_SET); */
 
-       pdb_set_hours(sampass, hours);
+       pdb_set_hours(sampass, hours, PDB_SET);
 
        return True;
 }
 
+static BOOL need_ldap_mod(BOOL pdb_add, const SAM_ACCOUNT * sampass, enum pdb_elements element) {
+       if (pdb_add) {
+               return (!IS_SAM_DEFAULT(sampass, element));
+       } else {
+               return IS_SAM_CHANGED(sampass, element);
+       }
+}
+
 /**********************************************************************
 Initialize SAM_ACCOUNT from an LDAP query
 (Based on init_buffer_from_sam in pdb_tdb.c)
 *********************************************************************/
 static BOOL init_ldap_from_sam (struct ldapsam_privates *ldap_state, 
                                LDAPMod *** mods, int ldap_op, 
+                               BOOL pdb_add,
                                const SAM_ACCOUNT * sampass)
 {
        pstring temp;
@@ -822,107 +1303,139 @@ static BOOL init_ldap_from_sam (struct ldapsam_privates *ldap_state,
         * took out adding "objectclass: sambaAccount"
         * do this on a per-mod basis
         */
-
-       make_a_mod(mods, ldap_op, "uid", pdb_get_username(sampass));
-       DEBUG(2, ("Setting entry for user: %s\n", pdb_get_username(sampass)));
-
-       if ( pdb_get_user_rid(sampass) ) {
-               rid = pdb_get_user_rid(sampass);
-       } else if (IS_SAM_SET(sampass, FLAG_SAM_UID)) {
+       if (need_ldap_mod(pdb_add, sampass, PDB_USERNAME)) {
+               make_a_mod(mods, ldap_op, "uid", pdb_get_username(sampass));
+               DEBUG(2, ("Setting entry for user: %s\n", pdb_get_username(sampass)));
+       }
+       
+       if ((rid = pdb_get_user_rid(sampass))!=0 ) {
+               if (need_ldap_mod(pdb_add, sampass, PDB_USERSID)) {             
+                       slprintf(temp, sizeof(temp) - 1, "%i", rid);
+                       make_a_mod(mods, ldap_op, "rid", temp);
+               }
+       } else if (!IS_SAM_DEFAULT(sampass, PDB_UID)) {
                rid = fallback_pdb_uid_to_user_rid(pdb_get_uid(sampass));
+               slprintf(temp, sizeof(temp) - 1, "%i", rid);
+               make_a_mod(mods, ldap_op, "rid", temp);
        } else if (ldap_state->permit_non_unix_accounts) {
                rid = ldapsam_get_next_available_nua_rid(ldap_state);
                if (rid == 0) {
                        DEBUG(0, ("NO user RID specified on account %s, and findining next available NUA RID failed, cannot store!\n", pdb_get_username(sampass)));
                        return False;
                }
+               slprintf(temp, sizeof(temp) - 1, "%i", rid);
+               make_a_mod(mods, ldap_op, "rid", temp);
        } else {
                DEBUG(0, ("NO user RID specified on account %s, cannot store!\n", pdb_get_username(sampass)));
                return False;
        }
 
-       slprintf(temp, sizeof(temp) - 1, "%i", rid);
-       make_a_mod(mods, ldap_op, "rid", temp);
 
-       if ( pdb_get_group_rid(sampass) ) {
-               rid = pdb_get_group_rid(sampass);
-       } else if (IS_SAM_SET(sampass, FLAG_SAM_GID)) {
+
+       if ((rid = pdb_get_group_rid(sampass))!=0 ) {
+               if (need_ldap_mod(pdb_add, sampass, PDB_GROUPSID)) {            
+                       slprintf(temp, sizeof(temp) - 1, "%i", rid);
+                       make_a_mod(mods, ldap_op, "primaryGroupID", temp);
+               }
+       } else if (!IS_SAM_DEFAULT(sampass, PDB_GID)) {
                rid = pdb_gid_to_group_rid(pdb_get_gid(sampass));
+               slprintf(temp, sizeof(temp) - 1, "%i", rid);
+               make_a_mod(mods, ldap_op, "primaryGroupID", temp);
        } else if (ldap_state->permit_non_unix_accounts) {
                rid = DOMAIN_GROUP_RID_USERS;
+               slprintf(temp, sizeof(temp) - 1, "%i", rid);
+               make_a_mod(mods, ldap_op, "primaryGroupID", temp);
        } else {
                DEBUG(0, ("NO group RID specified on account %s, cannot store!\n", pdb_get_username(sampass)));
                return False;
        }
 
-       slprintf(temp, sizeof(temp) - 1, "%i", rid);
-       make_a_mod(mods, ldap_op, "primaryGroupID", temp);
-
-       slprintf (temp, sizeof (temp) - 1, "%li", pdb_get_pass_last_set_time(sampass));
-       make_a_mod(mods, ldap_op, "pwdLastSet", temp);
 
        /* displayName, cn, and gecos should all be the same
-          *  most easily accomplished by giving them the same OID
-          *  gecos isn't set here b/c it should be handled by the 
-          *  add-user script
+        *  most easily accomplished by giving them the same OID
+        *  gecos isn't set here b/c it should be handled by the 
+        *  add-user script
         */
-
-       make_a_mod(mods, ldap_op, "displayName", pdb_get_fullname(sampass));
-       make_a_mod(mods, ldap_op, "cn", pdb_get_fullname(sampass));
-       make_a_mod(mods, ldap_op, "description", pdb_get_acct_desc(sampass));
-       make_a_mod(mods, ldap_op, "userWorkstations", pdb_get_workstations(sampass));
-
+       if (need_ldap_mod(pdb_add, sampass, PDB_FULLNAME)) {
+               make_a_mod(mods, ldap_op, "displayName", pdb_get_fullname(sampass));
+               make_a_mod(mods, ldap_op, "cn", pdb_get_fullname(sampass));
+       }
+       if (need_ldap_mod(pdb_add, sampass, PDB_ACCTDESC)) {    
+               make_a_mod(mods, ldap_op, "description", pdb_get_acct_desc(sampass));
+       }
+       if (need_ldap_mod(pdb_add, sampass, PDB_WORKSTATIONS)) {        
+               make_a_mod(mods, ldap_op, "userWorkstations", pdb_get_workstations(sampass));
+       }
        /*
         * Only updates fields which have been set (not defaults from smb.conf)
         */
 
-       if (IS_SAM_SET(sampass, FLAG_SAM_SMBHOME))
+       if (need_ldap_mod(pdb_add, sampass, PDB_SMBHOME)) {
                make_a_mod(mods, ldap_op, "smbHome", pdb_get_homedir(sampass));
-               
-       if (IS_SAM_SET(sampass, FLAG_SAM_DRIVE))
-               make_a_mod(mods, ldap_op, "homeDrive", pdb_get_dirdrive(sampass));
+       }
+                       
+       if (need_ldap_mod(pdb_add, sampass, PDB_DRIVE)) {
+               make_a_mod(mods, ldap_op, "homeDrive", pdb_get_dir_drive(sampass));
+       }
        
-       if (IS_SAM_SET(sampass, FLAG_SAM_LOGONSCRIPT))
+       if (need_ldap_mod(pdb_add, sampass, PDB_LOGONSCRIPT)) {
                make_a_mod(mods, ldap_op, "scriptPath", pdb_get_logon_script(sampass));
-
-       if (IS_SAM_SET(sampass, FLAG_SAM_PROFILE))
+       }
+       
+       if (need_ldap_mod(pdb_add, sampass, PDB_PROFILE))
                make_a_mod(mods, ldap_op, "profilePath", pdb_get_profile_path(sampass));
 
-       if (IS_SAM_SET(sampass, FLAG_SAM_LOGONTIME)) {
+       if (need_ldap_mod(pdb_add, sampass, PDB_LOGONTIME)) {
                slprintf(temp, sizeof(temp) - 1, "%li", pdb_get_logon_time(sampass));
                make_a_mod(mods, ldap_op, "logonTime", temp);
        }
 
-       if (IS_SAM_SET(sampass, FLAG_SAM_LOGOFFTIME)) {
+       if (need_ldap_mod(pdb_add, sampass, PDB_LOGOFFTIME)) {
                slprintf(temp, sizeof(temp) - 1, "%li", pdb_get_logoff_time(sampass));
                make_a_mod(mods, ldap_op, "logoffTime", temp);
        }
 
-       if (IS_SAM_SET(sampass, FLAG_SAM_KICKOFFTIME)) {
+       if (need_ldap_mod(pdb_add, sampass, PDB_KICKOFFTIME)) {
                slprintf (temp, sizeof (temp) - 1, "%li", pdb_get_kickoff_time(sampass));
                make_a_mod(mods, ldap_op, "kickoffTime", temp);
        }
 
-       if (IS_SAM_SET(sampass, FLAG_SAM_CANCHANGETIME)) {
+
+       if (need_ldap_mod(pdb_add, sampass, PDB_CANCHANGETIME)) {
                slprintf (temp, sizeof (temp) - 1, "%li", pdb_get_pass_can_change_time(sampass));
                make_a_mod(mods, ldap_op, "pwdCanChange", temp);
        }
 
-       if (IS_SAM_SET(sampass, FLAG_SAM_MUSTCHANGETIME)) {
+       if (need_ldap_mod(pdb_add, sampass, PDB_MUSTCHANGETIME)) {
                slprintf (temp, sizeof (temp) - 1, "%li", pdb_get_pass_must_change_time(sampass));
                make_a_mod(mods, ldap_op, "pwdMustChange", temp);
        }
 
+       if ((pdb_get_acct_ctrl(sampass)&(ACB_WSTRUST|ACB_SVRTRUST|ACB_DOMTRUST))||
+               (lp_ldap_passwd_sync()!=LDAP_PASSWD_SYNC_ONLY)) {
+
+               if (need_ldap_mod(pdb_add, sampass, PDB_LMPASSWD)) {
+                       pdb_sethexpwd (temp, pdb_get_lanman_passwd(sampass), pdb_get_acct_ctrl(sampass));
+                       make_a_mod (mods, ldap_op, "lmPassword", temp);
+               }
+               
+               if (need_ldap_mod(pdb_add, sampass, PDB_NTPASSWD)) {
+                       pdb_sethexpwd (temp, pdb_get_nt_passwd(sampass), pdb_get_acct_ctrl(sampass));
+                       make_a_mod (mods, ldap_op, "ntPassword", temp);
+               }
+               
+               if (need_ldap_mod(pdb_add, sampass, PDB_PASSLASTSET)) {
+                       slprintf (temp, sizeof (temp) - 1, "%li", pdb_get_pass_last_set_time(sampass));
+                       make_a_mod(mods, ldap_op, "pwdLastSet", temp);
+               }
+       }
+
        /* FIXME: Hours stuff goes in LDAP  */
-       pdb_sethexpwd (temp, pdb_get_lanman_passwd(sampass), pdb_get_acct_ctrl(sampass));
-       make_a_mod (mods, ldap_op, "lmPassword", temp);
-       
-       pdb_sethexpwd (temp, pdb_get_nt_passwd(sampass), pdb_get_acct_ctrl(sampass));
-       make_a_mod (mods, ldap_op, "ntPassword", temp);
+       if (need_ldap_mod(pdb_add, sampass, PDB_ACCTCTRL)) {
+               make_a_mod (mods, ldap_op, "acctFlags", pdb_encode_acct_ctrl (pdb_get_acct_ctrl(sampass),
+                       NEW_PW_FORMAT_SPACE_PADDED_LEN));
+       }
        
-       make_a_mod (mods, ldap_op, "acctFlags", pdb_encode_acct_ctrl (pdb_get_acct_ctrl(sampass),
-               NEW_PW_FORMAT_SPACE_PADDED_LEN));
-
        return True;
 }
 
@@ -930,7 +1443,7 @@ static BOOL init_ldap_from_sam (struct ldapsam_privates *ldap_state,
 /**********************************************************************
 Connect to LDAP server and find the next available RID.
 *********************************************************************/
-static uint32 check_nua_rid_is_avail(struct ldapsam_privates *ldap_state, uint32 top_rid, LDAP *ldap_struct
+static uint32 check_nua_rid_is_avail(struct ldapsam_privates *ldap_state, uint32 top_rid) 
 {
        LDAPMessage *result;
        uint32 final_rid = (top_rid & (~USER_RID_TYPE)) + RID_MULTIPLIER;
@@ -942,34 +1455,33 @@ static uint32 check_nua_rid_is_avail(struct ldapsam_privates *ldap_state, uint32
                return 0;
        }
 
-       if (ldapsam_search_one_user_by_rid(ldap_state, ldap_struct, final_rid, &result) != LDAP_SUCCESS) {
+       if (ldapsam_search_one_user_by_rid(ldap_state, final_rid, &result) != LDAP_SUCCESS) {
                DEBUG(0, ("Cannot allocate NUA RID %d (0x%x), as the confirmation search failed!\n", final_rid, final_rid));
-               final_rid = 0;
-               ldap_msgfree(result);
+               return 0;
        }
 
-       if (ldap_count_entries(ldap_struct, result) != 0)
-       {
+       if (ldap_count_entries(ldap_state->ldap_struct, result) != 0) {
                DEBUG(0, ("Cannot allocate NUA RID %d (0x%x), as the RID is already in use!!\n", final_rid, final_rid));
-               final_rid = 0;
                ldap_msgfree(result);
+               return 0;
        }
 
        DEBUG(5, ("NUA RID %d (0x%x), declared valid\n", final_rid, final_rid));
+       ldap_msgfree(result);
        return final_rid;
 }
 
 /**********************************************************************
 Extract the RID from an LDAP entry
 *********************************************************************/
-static uint32 entry_to_user_rid(struct ldapsam_privates *ldap_state, LDAPMessage *entry, LDAP *ldap_struct) {
+static uint32 entry_to_user_rid(struct ldapsam_privates *ldap_state, LDAPMessage *entry) {
        uint32 rid;
        SAM_ACCOUNT *user = NULL;
        if (!NT_STATUS_IS_OK(pdb_init_sam(&user))) {
                return 0;
        }
 
-       if (init_sam_from_ldap(ldap_state, user, ldap_struct, entry)) {
+       if (init_sam_from_ldap(ldap_state, user, entry)) {
                rid = pdb_get_user_rid(user);
        } else {
                rid =0;
@@ -985,7 +1497,7 @@ static uint32 entry_to_user_rid(struct ldapsam_privates *ldap_state, LDAPMessage
 /**********************************************************************
 Connect to LDAP server and find the next available RID.
 *********************************************************************/
-static uint32 search_top_nua_rid(struct ldapsam_privates *ldap_state, LDAP *ldap_struct)
+static uint32 search_top_nua_rid(struct ldapsam_privates *ldap_state)
 {
        int rc;
        pstring filter;
@@ -1006,23 +1518,20 @@ static uint32 search_top_nua_rid(struct ldapsam_privates *ldap_state, LDAP *ldap
 #endif 
        DEBUG(2, ("ldapsam_get_next_available_nua_rid: searching for:[%s]\n", final_filter));
 
-       rc = ldap_search_s(ldap_struct, lp_ldap_suffix(),
+       rc = ldapsam_search(ldap_state, lp_ldap_suffix(),
                           LDAP_SCOPE_SUBTREE, final_filter, attr, 0,
                           &result);
 
-       if (rc != LDAP_SUCCESS)
-       {
-               
+       if (rc != LDAP_SUCCESS) {
                DEBUG(3, ("LDAP search failed! cannot find base for NUA RIDs: %s\n", ldap_err2string(rc)));
                DEBUGADD(3, ("Query was: %s, %s\n", lp_ldap_suffix(), final_filter));
 
                free(final_filter);
-               ldap_msgfree(result);
                result = NULL;
                return 0;
        }
        
-       count = ldap_count_entries(ldap_struct, result);
+       count = ldap_count_entries(ldap_state->ldap_struct, result);
        DEBUG(2, ("search_top_nua_rid: %d entries in the base!\n", count));
        
        if (count == 0) {
@@ -1035,13 +1544,13 @@ static uint32 search_top_nua_rid(struct ldapsam_privates *ldap_state, LDAP *ldap
        }
        
        free(final_filter);
-       entry = ldap_first_entry(ldap_struct,result);
+       entry = ldap_first_entry(ldap_state->ldap_struct,result);
 
-       top_rid = entry_to_user_rid(ldap_state, entry, ldap_struct);
+       top_rid = entry_to_user_rid(ldap_state, entry);
 
-       while ((entry = ldap_next_entry(ldap_struct, entry))) {
+       while ((entry = ldap_next_entry(ldap_state->ldap_struct, entry))) {
 
-               rid = entry_to_user_rid(ldap_state, entry, ldap_struct);
+               rid = entry_to_user_rid(ldap_state, entry);
                if (rid > top_rid) {
                        top_rid = rid;
                }
@@ -1059,64 +1568,39 @@ static uint32 search_top_nua_rid(struct ldapsam_privates *ldap_state, LDAP *ldap
 Connect to LDAP server and find the next available RID.
 *********************************************************************/
 static uint32 ldapsam_get_next_available_nua_rid(struct ldapsam_privates *ldap_state) {
-       LDAP *ldap_struct;
        uint32 next_nua_rid;
        uint32 top_nua_rid;
 
-       if (!ldapsam_open_connection(ldap_state, &ldap_struct))
-       {
-               return 0;
-       }
-       if (!ldapsam_connect_system(ldap_state, ldap_struct))
-       {
-               ldap_unbind(ldap_struct);
-               return 0;
-       }
-       
-       top_nua_rid = search_top_nua_rid(ldap_state, ldap_struct);
+       top_nua_rid = search_top_nua_rid(ldap_state);
 
        next_nua_rid = check_nua_rid_is_avail(ldap_state, 
-                                             top_nua_rid, ldap_struct);
+                                             top_nua_rid);
        
-       ldap_unbind(ldap_struct);
        return next_nua_rid;
 }
 
 /**********************************************************************
 Connect to LDAP server for password enumeration
 *********************************************************************/
-static BOOL ldapsam_setsampwent(struct pdb_methods *my_methods, BOOL update)
+static NTSTATUS ldapsam_setsampwent(struct pdb_methods *my_methods, BOOL update)
 {
        struct ldapsam_privates *ldap_state = (struct ldapsam_privates *)my_methods->private_data;
        int rc;
        pstring filter;
 
-       if (!ldapsam_open_connection(ldap_state, &ldap_state->ldap_struct))
-       {
-               return False;
-       }
-       if (!ldapsam_connect_system(ldap_state, ldap_state->ldap_struct))
-       {
-               ldap_unbind(ldap_state->ldap_struct);
-               return False;
-       }
-
        pstrcpy(filter, lp_ldap_filter());
        all_string_sub(filter, "%u", "*", sizeof(pstring));
 
-       rc = ldap_search_s(ldap_state->ldap_struct, lp_ldap_suffix(),
+       rc = ldapsam_search(ldap_state, lp_ldap_suffix(),
                           LDAP_SCOPE_SUBTREE, filter, attr, 0,
                           &ldap_state->result);
 
-       if (rc != LDAP_SUCCESS)
-       {
+       if (rc != LDAP_SUCCESS) {
                DEBUG(0, ("LDAP search failed: %s\n", ldap_err2string(rc)));
                DEBUG(3, ("Query was: %s, %s\n", lp_ldap_suffix(), filter));
                ldap_msgfree(ldap_state->result);
-               ldap_unbind(ldap_state->ldap_struct);
-               ldap_state->ldap_struct = NULL;
                ldap_state->result = NULL;
-               return False;
+               return NT_STATUS_UNSUCCESSFUL;
        }
 
        DEBUG(2, ("ldapsam_setsampwent: %d entries in the base!\n",
@@ -1127,7 +1611,7 @@ static BOOL ldapsam_setsampwent(struct pdb_methods *my_methods, BOOL update)
                                 ldap_state->result);
        ldap_state->index = 0;
 
-       return True;
+       return NT_STATUS_OK;
 }
 
 /**********************************************************************
@@ -1136,11 +1620,8 @@ End enumeration of the LDAP password list
 static void ldapsam_endsampwent(struct pdb_methods *my_methods)
 {
        struct ldapsam_privates *ldap_state = (struct ldapsam_privates *)my_methods->private_data;
-       if (ldap_state->ldap_struct && ldap_state->result)
-       {
+       if (ldap_state->result) {
                ldap_msgfree(ldap_state->result);
-               ldap_unbind(ldap_state->ldap_struct);
-               ldap_state->ldap_struct = NULL;
                ldap_state->result = NULL;
        }
 }
@@ -1148,319 +1629,357 @@ static void ldapsam_endsampwent(struct pdb_methods *my_methods)
 /**********************************************************************
 Get the next entry in the LDAP password database 
 *********************************************************************/
-static BOOL ldapsam_getsampwent(struct pdb_methods *my_methods, SAM_ACCOUNT * user)
+static NTSTATUS ldapsam_getsampwent(struct pdb_methods *my_methods, SAM_ACCOUNT *user)
 {
+       NTSTATUS ret = NT_STATUS_UNSUCCESSFUL;
        struct ldapsam_privates *ldap_state = (struct ldapsam_privates *)my_methods->private_data;
-       BOOL ret = False;
+       BOOL bret = False;
 
-       while (!ret) {
+       /* The rebind proc needs this *HACK*.  We are not multithreaded, so
+          this will work, but it's not nice. */
+       static_ldap_state = ldap_state;
+
+       while (!bret) {
                if (!ldap_state->entry)
-                       return False;
+                       return ret;
                
                ldap_state->index++;
-               ret = init_sam_from_ldap(ldap_state, user, ldap_state->ldap_struct,
-                                        ldap_state->entry);
+               bret = init_sam_from_ldap(ldap_state, user, ldap_state->entry);
                
                ldap_state->entry = ldap_next_entry(ldap_state->ldap_struct,
-                                           ldap_state->entry);
-               
+                                           ldap_state->entry); 
        }
 
-       return True;
+       return NT_STATUS_OK;
 }
 
 /**********************************************************************
 Get SAM_ACCOUNT entry from LDAP by username 
 *********************************************************************/
-static BOOL ldapsam_getsampwnam(struct pdb_methods *my_methods, SAM_ACCOUNT * user, const char *sname)
+static NTSTATUS ldapsam_getsampwnam(struct pdb_methods *my_methods, SAM_ACCOUNT *user, const char *sname)
 {
+       NTSTATUS ret = NT_STATUS_UNSUCCESSFUL;
        struct ldapsam_privates *ldap_state = (struct ldapsam_privates *)my_methods->private_data;
-       LDAP *ldap_struct;
        LDAPMessage *result;
        LDAPMessage *entry;
-
-       if (!ldapsam_open_connection(ldap_state, &ldap_struct))
-               return False;
-       if (!ldapsam_connect_system(ldap_state, ldap_struct))
-       {
-               ldap_unbind(ldap_struct);
-               return False;
-       }
-       if (ldapsam_search_one_user_by_name(ldap_state, ldap_struct, sname, &result) != LDAP_SUCCESS)
-       {
-               ldap_unbind(ldap_struct);
-               return False;
+       int count;
+       
+       if (ldapsam_search_one_user_by_name(ldap_state, sname, &result) != LDAP_SUCCESS) {
+               return NT_STATUS_NO_SUCH_USER;
        }
-       if (ldap_count_entries(ldap_struct, result) < 1)
-       {
+       
+       count = ldap_count_entries(ldap_state->ldap_struct, result);
+       
+       if (count < 1) {
                DEBUG(4,
                      ("We don't find this user [%s] count=%d\n", sname,
-                      ldap_count_entries(ldap_struct, result)));
-               ldap_unbind(ldap_struct);
-               return False;
-       }
-       entry = ldap_first_entry(ldap_struct, result);
-       if (entry)
-       {
-               if (!init_sam_from_ldap(ldap_state, user, ldap_struct, entry)) {
-                       DEBUG(0,("ldapsam_getsampwnam: init_sam_from_ldap failed!\n"));
+                      count));
+               return NT_STATUS_NO_SUCH_USER;
+       } else if (count > 1) {
+               DEBUG(1,
+                     ("Duplicate entries for this user [%s] Failing. count=%d\n", sname,
+                      count));
+               return NT_STATUS_NO_SUCH_USER;
+       }
+
+       entry = ldap_first_entry(ldap_state->ldap_struct, result);
+       if (entry) {
+               if (!init_sam_from_ldap(ldap_state, user, entry)) {
+                       DEBUG(1,("ldapsam_getsampwnam: init_sam_from_ldap failed for user '%s'!\n", sname));
                        ldap_msgfree(result);
-                       ldap_unbind(ldap_struct);
-                       return False;
+                       return NT_STATUS_NO_SUCH_USER;
                }
                ldap_msgfree(result);
-               ldap_unbind(ldap_struct);
-               return True;
-       }
-       else
-       {
+               ret = NT_STATUS_OK;
+       } else {
                ldap_msgfree(result);
-               ldap_unbind(ldap_struct);
-               return False;
        }
+       return ret;
 }
 
 /**********************************************************************
 Get SAM_ACCOUNT entry from LDAP by rid 
 *********************************************************************/
-static BOOL ldapsam_getsampwrid(struct pdb_methods *my_methods, SAM_ACCOUNT * user, uint32 rid)
+static NTSTATUS ldapsam_getsampwrid(struct pdb_methods *my_methods, SAM_ACCOUNT *user, uint32 rid)
 {
-       struct ldapsam_privates *ldap_state = (struct ldapsam_privates *)my_methods->private_data;
-       LDAP *ldap_struct;
+       NTSTATUS ret = NT_STATUS_UNSUCCESSFUL;
+       struct ldapsam_privates *ldap_state = 
+               (struct ldapsam_privates *)my_methods->private_data;
        LDAPMessage *result;
        LDAPMessage *entry;
+       int count;
 
-       if (!ldapsam_open_connection(ldap_state, &ldap_struct))
-               return False;
-
-       if (!ldapsam_connect_system(ldap_state, ldap_struct))
-       {
-               ldap_unbind(ldap_struct);
-               return False;
-       }
-       if (ldapsam_search_one_user_by_rid(ldap_state, ldap_struct, rid, &result) !=
-           LDAP_SUCCESS)
-       {
-               ldap_unbind(ldap_struct);
-               return False;
+       if (ldapsam_search_one_user_by_rid(ldap_state, rid, &result) != LDAP_SUCCESS) {
+               return NT_STATUS_NO_SUCH_USER;
        }
 
-       if (ldap_count_entries(ldap_struct, result) < 1)
-       {
-               DEBUG(0,
+       count = ldap_count_entries(ldap_state->ldap_struct, result);
+               
+       if (count < 1) {
+               DEBUG(4,
                      ("We don't find this rid [%i] count=%d\n", rid,
-                      ldap_count_entries(ldap_struct, result)));
-               ldap_unbind(ldap_struct);
-               return False;
-       }
-
-       entry = ldap_first_entry(ldap_struct, result);
-       if (entry)
-       {
-               if (!init_sam_from_ldap(ldap_state, user, ldap_struct, entry)) {
-                       DEBUG(0,("ldapsam_getsampwrid: init_sam_from_ldap failed!\n"));
+                      count));
+               return NT_STATUS_NO_SUCH_USER;
+       } else if (count > 1) {
+               DEBUG(1,
+                     ("More than one user with rid [%i]. Failing. count=%d\n", rid,
+                      count));
+               return NT_STATUS_NO_SUCH_USER;
+       }
+
+       entry = ldap_first_entry(ldap_state->ldap_struct, result);
+       if (entry) {
+               if (!init_sam_from_ldap(ldap_state, user, entry)) {
+                       DEBUG(1,("ldapsam_getsampwrid: init_sam_from_ldap failed!\n"));
                        ldap_msgfree(result);
-                       ldap_unbind(ldap_struct);
-                       return False;
+                       return NT_STATUS_NO_SUCH_USER;
                }
                ldap_msgfree(result);
-               ldap_unbind(ldap_struct);
-               return True;
-       }
-       else
-       {
+               ret = NT_STATUS_OK;
+       } else {
                ldap_msgfree(result);
-               ldap_unbind(ldap_struct);
-               return False;
        }
+       return ret;
 }
 
-static BOOL ldapsam_getsampwsid(struct pdb_methods *my_methods, SAM_ACCOUNT * user, DOM_SID *sid)
+static NTSTATUS ldapsam_getsampwsid(struct pdb_methods *my_methods, SAM_ACCOUNT * user, const DOM_SID *sid)
 {
        uint32 rid;
        if (!sid_peek_check_rid(get_global_sam_sid(), sid, &rid))
-               return False;
+               return NT_STATUS_NO_SUCH_USER;
        return ldapsam_getsampwrid(my_methods, user, rid);
 }      
 
+/********************************************************************
+Do the actual modification - also change a plaittext passord if 
+it it set.
+**********************************************************************/
+
+static NTSTATUS ldapsam_modify_entry(struct pdb_methods *my_methods, 
+                                    SAM_ACCOUNT *newpwd, char *dn,
+                                    LDAPMod **mods, int ldap_op, BOOL pdb_add)
+{
+       struct ldapsam_privates *ldap_state = (struct ldapsam_privates *)my_methods->private_data;
+       int rc;
+       
+       if (!my_methods || !newpwd || !dn) {
+               return NT_STATUS_INVALID_PARAMETER;
+       }
+       
+       if (!mods) {
+               DEBUG(5,("mods is empty: nothing to modify\n"));
+               /* may be password change below however */
+       } else {
+               switch(ldap_op)
+               {
+                       case LDAP_MOD_ADD: 
+                               make_a_mod(&mods, LDAP_MOD_ADD, "objectclass", "account");
+                               rc = ldapsam_add(ldap_state, dn, mods);
+                               break;
+                       case LDAP_MOD_REPLACE: 
+                               rc = ldapsam_modify(ldap_state, dn ,mods);
+                               break;
+                       default:        
+                               DEBUG(0,("Wrong LDAP operation type: %d!\n", ldap_op));
+                               return NT_STATUS_UNSUCCESSFUL;
+               }
+               
+               if (rc!=LDAP_SUCCESS) {
+                       char *ld_error;
+                       ldap_get_option(ldap_state->ldap_struct, LDAP_OPT_ERROR_STRING,
+                                       &ld_error);
+                       DEBUG(1,
+                             ("failed to %s user dn= %s with: %s\n\t%s\n",
+                              ldap_op == LDAP_MOD_ADD ? "add" : "modify",
+                              dn, ldap_err2string(rc),
+                              ld_error));
+                       free(ld_error);
+                       return NT_STATUS_UNSUCCESSFUL;
+               }  
+       }
+       
+#ifdef LDAP_EXOP_X_MODIFY_PASSWD
+       if (!(pdb_get_acct_ctrl(newpwd)&(ACB_WSTRUST|ACB_SVRTRUST|ACB_DOMTRUST))&&
+               (lp_ldap_passwd_sync()!=LDAP_PASSWD_SYNC_OFF)&&
+               need_ldap_mod(pdb_add, newpwd, PDB_PLAINTEXT_PW)&&
+               (pdb_get_plaintext_passwd(newpwd)!=NULL)) {
+               BerElement *ber;
+               struct berval *bv;
+               char *retoid;
+               struct berval *retdata;
+
+               if ((ber = ber_alloc_t(LBER_USE_DER))==NULL) {
+                       DEBUG(0,("ber_alloc_t returns NULL\n"));
+                       return NT_STATUS_UNSUCCESSFUL;
+               }
+               ber_printf (ber, "{");
+               ber_printf (ber, "ts", LDAP_TAG_EXOP_X_MODIFY_PASSWD_ID,dn);
+               ber_printf (ber, "ts", LDAP_TAG_EXOP_X_MODIFY_PASSWD_NEW, pdb_get_plaintext_passwd(newpwd));
+               ber_printf (ber, "N}");
+
+               if ((rc = ber_flatten (ber, &bv))<0) {
+                       DEBUG(0,("ber_flatten returns a value <0\n"));
+                       return NT_STATUS_UNSUCCESSFUL;
+               }
+               
+               ber_free(ber,1);
+
+               if ((rc = ldapsam_extended_operation(ldap_state, LDAP_EXOP_X_MODIFY_PASSWD,
+                                                   bv, NULL, NULL, &retoid, &retdata))!=LDAP_SUCCESS) {
+                       DEBUG(0,("LDAP Password could not be changed for user %s: %s\n",
+                               pdb_get_username(newpwd),ldap_err2string(rc)));
+               } else {
+                       DEBUG(3,("LDAP Password changed for user %s\n",pdb_get_username(newpwd)));
+    
+                       ber_bvfree(retdata);
+                       ber_memfree(retoid);
+               }
+               ber_bvfree(bv);
+       }
+#else
+       DEBUG(10,("LDAP PASSWORD SYNC is not supported!\n"));
+#endif /* LDAP_EXOP_X_MODIFY_PASSWD */
+       return NT_STATUS_OK;
+}
+
 /**********************************************************************
 Delete entry from LDAP for username 
 *********************************************************************/
-static BOOL ldapsam_delete_sam_account(struct pdb_methods *my_methods, SAM_ACCOUNT * sam_acct)
+static NTSTATUS ldapsam_delete_sam_account(struct pdb_methods *my_methods, SAM_ACCOUNT * sam_acct)
 {
        struct ldapsam_privates *ldap_state = (struct ldapsam_privates *)my_methods->private_data;
        const char *sname;
        int rc;
-       char *dn;
-       LDAP *ldap_struct;
-       LDAPMessage *entry;
        LDAPMessage *result;
+       NTSTATUS ret;
+       const char *sam_user_attrs[] =
+       { "lmPassword", "ntPassword", "pwdLastSet", "logonTime", "logoffTime",
+         "kickoffTime", "pwdCanChange", "pwdMustChange", "acctFlags",
+         "displayName", "smbHome", "homeDrive", "scriptPath", "profilePath",
+         "userWorkstations", "primaryGroupID", "domain", "rid", NULL };
 
        if (!sam_acct) {
                DEBUG(0, ("sam_acct was NULL!\n"));
-               return False;
+               return NT_STATUS_INVALID_PARAMETER;
        }
 
        sname = pdb_get_username(sam_acct);
 
-       if (!ldapsam_open_connection(ldap_state, &ldap_struct))
-               return False;
-
        DEBUG (3, ("Deleting user %s from LDAP.\n", sname));
-       
-       if (!ldapsam_connect_system(ldap_state, ldap_struct)) {
-               ldap_unbind (ldap_struct);
-               DEBUG(0, ("Failed to delete user %s from LDAP.\n", sname));
-               return False;
-       }
-
-       rc = ldapsam_search_one_user_by_name(ldap_state, ldap_struct, sname, &result);
-       if (ldap_count_entries (ldap_struct, result) == 0) {
-               DEBUG (0, ("User doesn't exit!\n"));
-               ldap_msgfree (result);
-               ldap_unbind (ldap_struct);
-               return False;
-       }
-
-       entry = ldap_first_entry (ldap_struct, result);
-       dn = ldap_get_dn (ldap_struct, entry);
-
-       rc = ldap_delete_s (ldap_struct, dn);
 
-       ldap_memfree (dn);
+       rc = ldapsam_search_one_user_by_name(ldap_state, sname, &result);
        if (rc != LDAP_SUCCESS) {
-               char *ld_error;
-               ldap_get_option (ldap_struct, LDAP_OPT_ERROR_STRING, &ld_error);
-               DEBUG (0,("failed to delete user with uid = %s with: %s\n\t%s\n",
-                       sname, ldap_err2string (rc), ld_error));
-               free (ld_error);
-               ldap_unbind (ldap_struct);
-               return False;
+               return NT_STATUS_NO_SUCH_USER;
        }
 
-       DEBUG (2,("successfully deleted uid = %s from the LDAP database\n", sname));
-       ldap_unbind (ldap_struct);
-       return True;
+       ret = ldapsam_delete_entry(ldap_state, result, "sambaAccount",
+                                  sam_user_attrs);
+       ldap_msgfree(result);
+       return ret;
 }
 
 /**********************************************************************
 Update SAM_ACCOUNT 
 *********************************************************************/
-static BOOL ldapsam_update_sam_account(struct pdb_methods *my_methods, SAM_ACCOUNT * newpwd)
+static NTSTATUS ldapsam_update_sam_account(struct pdb_methods *my_methods, SAM_ACCOUNT * newpwd)
 {
+       NTSTATUS ret = NT_STATUS_UNSUCCESSFUL;
        struct ldapsam_privates *ldap_state = (struct ldapsam_privates *)my_methods->private_data;
        int rc;
        char *dn;
-       LDAP *ldap_struct;
        LDAPMessage *result;
        LDAPMessage *entry;
        LDAPMod **mods;
 
-       if (!ldapsam_open_connection(ldap_state, &ldap_struct)) /* open a connection to the server */
-               return False;
-
-       if (!ldapsam_connect_system(ldap_state, ldap_struct))   /* connect as system account */
-       {
-               ldap_unbind(ldap_struct);
-               return False;
+       if (!init_ldap_from_sam(ldap_state, &mods, LDAP_MOD_REPLACE, False, newpwd)) {
+               DEBUG(0, ("ldapsam_update_sam_account: init_ldap_from_sam failed!\n"));
+               return NT_STATUS_UNSUCCESSFUL;
+       }
+       
+       if (mods == NULL) {
+               DEBUG(4,("mods is empty: nothing to update for user: %s\n",pdb_get_username(newpwd)));
+               return NT_STATUS_OK;
+       }
+       
+       rc = ldapsam_search_one_user_by_name(ldap_state, pdb_get_username(newpwd), &result);
+       if (rc != LDAP_SUCCESS) {
+               ldap_mods_free(mods, 1);
+               return NT_STATUS_UNSUCCESSFUL;
        }
 
-       rc = ldapsam_search_one_user_by_name(ldap_state, ldap_struct,
-                                            pdb_get_username(newpwd), &result);
-
-       if (ldap_count_entries(ldap_struct, result) == 0)
-       {
+       if (ldap_count_entries(ldap_state->ldap_struct, result) == 0) {
                DEBUG(0, ("No user to modify!\n"));
                ldap_msgfree(result);
-               ldap_unbind(ldap_struct);
-               return False;
+               ldap_mods_free(mods, 1);
+               return NT_STATUS_UNSUCCESSFUL;
        }
 
-       if (!init_ldap_from_sam(ldap_state, &mods, LDAP_MOD_REPLACE, newpwd)) {
-               DEBUG(0, ("ldapsam_update_sam_account: init_ldap_from_sam failed!\n"));
-               ldap_msgfree(result);
-               ldap_unbind(ldap_struct);
-               return False;
+       entry = ldap_first_entry(ldap_state->ldap_struct, result);
+       dn = ldap_get_dn(ldap_state->ldap_struct, entry);
+        ldap_msgfree(result);
+       
+       ret = ldapsam_modify_entry(my_methods,newpwd,dn,mods,LDAP_MOD_REPLACE, False);
+       if (NT_STATUS_IS_ERR(ret)) {
+               DEBUG(0,("failed to modify user with uid = %s\n",
+                                       pdb_get_username(newpwd)));
+               ldap_mods_free(mods,1);
+               return ret;
        }
 
-       entry = ldap_first_entry(ldap_struct, result);
-       dn = ldap_get_dn(ldap_struct, entry);
-
-       rc = ldap_modify_s(ldap_struct, dn, mods);
-
-       if (rc != LDAP_SUCCESS)
-       {
-               char *ld_error;
-               ldap_get_option(ldap_struct, LDAP_OPT_ERROR_STRING,
-                               &ld_error);
-               DEBUG(0,
-                     ("failed to modify user with uid = %s with: %s\n\t%s\n",
-                      pdb_get_username(newpwd), ldap_err2string(rc),
-                      ld_error));
-               free(ld_error);
-               ldap_unbind(ldap_struct);
-               return False;
-       }
 
        DEBUG(2,
              ("successfully modified uid = %s in the LDAP database\n",
               pdb_get_username(newpwd)));
        ldap_mods_free(mods, 1);
-       ldap_unbind(ldap_struct);
-       return True;
+       return NT_STATUS_OK;
 }
 
 /**********************************************************************
 Add SAM_ACCOUNT to LDAP 
 *********************************************************************/
-static BOOL ldapsam_add_sam_account(struct pdb_methods *my_methods, SAM_ACCOUNT * newpwd)
+static NTSTATUS ldapsam_add_sam_account(struct pdb_methods *my_methods, SAM_ACCOUNT * newpwd)
 {
+       NTSTATUS ret = NT_STATUS_UNSUCCESSFUL;
        struct ldapsam_privates *ldap_state = (struct ldapsam_privates *)my_methods->private_data;
        int rc;
        pstring filter;
-       LDAP *ldap_struct = NULL;
        LDAPMessage *result = NULL;
        pstring dn;
        LDAPMod **mods = NULL;
        int             ldap_op;
        uint32          num_result;
-
+       
        const char *username = pdb_get_username(newpwd);
        if (!username || !*username) {
                DEBUG(0, ("Cannot add user without a username!\n"));
-               return False;
+               return NT_STATUS_INVALID_PARAMETER;
        }
 
-       if (!ldapsam_open_connection(ldap_state, &ldap_struct)) /* open a connection to the server */
-       {
-               return False;
-       }
-
-       if (!ldapsam_connect_system(ldap_state, ldap_struct))   /* connect as system account */
-       {
-               ldap_unbind(ldap_struct);
-               return False;
+       rc = ldapsam_search_one_user_by_name (ldap_state, username, &result);
+       if (rc != LDAP_SUCCESS) {
+               return NT_STATUS_UNSUCCESSFUL;
        }
 
-       rc = ldapsam_search_one_user_by_name (ldap_state, ldap_struct, username, &result);
-
-       if (ldap_count_entries(ldap_struct, result) != 0)
-       {
-               DEBUG(0,("User already in the base, with samba properties\n"));
+       if (ldap_count_entries(ldap_state->ldap_struct, result) != 0) {
+               DEBUG(0,("User '%s' already in the base, with samba properties\n", 
+                        username));
                ldap_msgfree(result);
-               ldap_unbind(ldap_struct);
-               return False;
+               return NT_STATUS_UNSUCCESSFUL;
        }
        ldap_msgfree(result);
 
        slprintf (filter, sizeof (filter) - 1, "uid=%s", username);
-       rc = ldapsam_search_one_user(ldap_state, ldap_struct, filter, &result);
-       num_result = ldap_count_entries(ldap_struct, result);
+       rc = ldapsam_search_one_user(ldap_state, filter, &result);
+       if (rc != LDAP_SUCCESS) {
+               return NT_STATUS_UNSUCCESSFUL;
+       }
+
+       num_result = ldap_count_entries(ldap_state->ldap_struct, result);
        
        if (num_result > 1) {
                DEBUG (0, ("More than one user with that uid exists: bailing out!\n"));
                ldap_msgfree(result);
-               return False;
+               return NT_STATUS_UNSUCCESSFUL;
        }
        
        /* Check if we need to update an existing entry */
@@ -1470,12 +1989,11 @@ static BOOL ldapsam_add_sam_account(struct pdb_methods *my_methods, SAM_ACCOUNT
                
                DEBUG(3,("User exists without samba properties: adding them\n"));
                ldap_op = LDAP_MOD_REPLACE;
-               entry = ldap_first_entry (ldap_struct, result);
-               tmp = ldap_get_dn (ldap_struct, entry);
+               entry = ldap_first_entry (ldap_state->ldap_struct, result);
+               tmp = ldap_get_dn (ldap_state->ldap_struct, entry);
                slprintf (dn, sizeof (dn) - 1, "%s", tmp);
                ldap_memfree (tmp);
-       }
-       else {
+       } else {
                /* Check if we need to add an entry */
                DEBUG(3,("Adding new user\n"));
                ldap_op = LDAP_MOD_ADD;
@@ -1488,53 +2006,518 @@ static BOOL ldapsam_add_sam_account(struct pdb_methods *my_methods, SAM_ACCOUNT
 
        ldap_msgfree(result);
 
-       if (!init_ldap_from_sam(ldap_state, &mods, ldap_op, newpwd)) {
+       if (!init_ldap_from_sam(ldap_state, &mods, ldap_op, True, newpwd)) {
                DEBUG(0, ("ldapsam_add_sam_account: init_ldap_from_sam failed!\n"));
                ldap_mods_free(mods, 1);
-               ldap_unbind(ldap_struct);
-               return False;           
+               return NT_STATUS_UNSUCCESSFUL;          
        }
+       
+       if (mods == NULL) {
+               DEBUG(0,("mods is empty: nothing to add for user: %s\n",pdb_get_username(newpwd)));
+               return NT_STATUS_UNSUCCESSFUL;
+       }
+       
        make_a_mod(&mods, LDAP_MOD_ADD, "objectclass", "sambaAccount");
 
-       if (ldap_op == LDAP_MOD_REPLACE) {
-               rc = ldap_modify_s(ldap_struct, dn, mods);
-       }
-       else {
-               rc = ldap_add_s(ldap_struct, dn, mods);
+       ret = ldapsam_modify_entry(my_methods,newpwd,dn,mods,ldap_op, True);
+       if (NT_STATUS_IS_ERR(ret)) {
+               DEBUG(0,("failed to modify/add user with uid = %s (dn = %s)\n",
+                        pdb_get_username(newpwd),dn));
+               ldap_mods_free(mods,1);
+               return ret;
        }
 
-       if (rc != LDAP_SUCCESS)
-       {
-               char *ld_error;
-
-               ldap_get_option (ldap_struct, LDAP_OPT_ERROR_STRING, &ld_error);
-               DEBUG(0,("failed to modify/add user with uid = %s (dn = %s) with: %s\n\t%s\n",
-                       pdb_get_username(newpwd), dn, ldap_err2string (rc), ld_error));
-               free(ld_error);
-               ldap_mods_free(mods, 1);
-               ldap_unbind(ldap_struct);
-               return False;
-       }
-       
        DEBUG(2,("added: uid = %s in the LDAP database\n", pdb_get_username(newpwd)));
        ldap_mods_free(mods, 1);
-       ldap_unbind(ldap_struct);
-       return True;
+       return NT_STATUS_OK;
 }
 
 static void free_private_data(void **vp) 
 {
        struct ldapsam_privates **ldap_state = (struct ldapsam_privates **)vp;
 
-       if ((*ldap_state)->ldap_struct) {
-               ldap_unbind((*ldap_state)->ldap_struct);
+       ldapsam_close(*ldap_state);
+
+       if ((*ldap_state)->bind_secret) {
+               memset((*ldap_state)->bind_secret, '\0', strlen((*ldap_state)->bind_secret));
        }
 
+       ldapsam_close(*ldap_state);
+               
+       SAFE_FREE((*ldap_state)->bind_dn);
+       SAFE_FREE((*ldap_state)->bind_secret);
+
        *ldap_state = NULL;
 
        /* No need to free any further, as it is talloc()ed */
 }
 
+static const char *group_attr[] = {"gid", "ntSid", "ntGroupType",
+                                  "gidNumber",
+                                  "displayName", "description",
+                                  NULL };
+                                  
+static int ldapsam_search_one_group (struct ldapsam_privates *ldap_state,
+                                    const char *filter,
+                                    LDAPMessage ** result)
+{
+       int scope = LDAP_SCOPE_SUBTREE;
+       int rc;
+
+       DEBUG(2, ("ldapsam_search_one_group: searching for:[%s]\n", filter));
+
+       rc = ldapsam_search(ldap_state, lp_ldap_suffix (), scope,
+                           filter, group_attr, 0, result);
+
+       if (rc != LDAP_SUCCESS) {
+               DEBUG(0, ("ldapsam_search_one_group: "
+                         "Problem during the LDAP search: %s\n",
+                         ldap_err2string(rc)));
+               DEBUG(3, ("ldapsam_search_one_group: Query was: %s, %s\n",
+                         lp_ldap_suffix(), filter));
+       }
+
+       return rc;
+}
+
+static BOOL init_group_from_ldap(struct ldapsam_privates *ldap_state,
+                                GROUP_MAP *map, LDAPMessage *entry)
+{
+       pstring temp;
+
+       if (ldap_state == NULL || map == NULL || entry == NULL ||
+           ldap_state->ldap_struct == NULL) {
+               DEBUG(0, ("init_group_from_ldap: NULL parameters found!\n"));
+               return False;
+       }
+
+       if (!get_single_attribute(ldap_state->ldap_struct, entry, "gidNumber",
+                                 temp)) {
+               DEBUG(0, ("Mandatory attribute gidNumber not found\n"));
+               return False;
+       }
+       DEBUG(2, ("Entry found for group: %s\n", temp));
+
+       map->gid = (uint32)atol(temp);
+
+       if (!get_single_attribute(ldap_state->ldap_struct, entry, "ntSid",
+                                 temp)) {
+               DEBUG(0, ("Mandatory attribute ntSid not found\n"));
+               return False;
+       }
+       string_to_sid(&map->sid, temp);
+
+       if (!get_single_attribute(ldap_state->ldap_struct, entry, "ntGroupType",
+                                 temp)) {
+               DEBUG(0, ("Mandatory attribute ntGroupType not found\n"));
+               return False;
+       }
+       map->sid_name_use = (uint32)atol(temp);
+
+       if ((map->sid_name_use < SID_NAME_USER) ||
+           (map->sid_name_use > SID_NAME_UNKNOWN)) {
+               DEBUG(0, ("Unknown Group type: %d\n", map->sid_name_use));
+               return False;
+       }
+
+       if (!get_single_attribute(ldap_state->ldap_struct, entry, "displayName",
+                                 temp)) {
+               DEBUG(3, ("Attribute displayName not found\n"));
+               temp[0] = '\0';
+       }
+       fstrcpy(map->nt_name, temp);
+
+       if (!get_single_attribute(ldap_state->ldap_struct, entry, "description",
+                                 temp)) {
+               DEBUG(3, ("Attribute description not found\n"));
+               temp[0] = '\0';
+       }
+       fstrcpy(map->comment, temp);
+
+       map->systemaccount = 0;
+       init_privilege(&map->priv_set);
+
+       return True;
+}
+
+static BOOL init_ldap_from_group(struct ldapsam_privates *ldap_state,
+                                LDAPMod ***mods, int ldap_op,
+                                const GROUP_MAP *map)
+{
+       pstring tmp;
+
+       if (mods == NULL || map == NULL) {
+               DEBUG(0, ("init_ldap_from_group: NULL parameters found!\n"));
+               return False;
+       }
+
+       *mods = NULL;
+
+       sid_to_string(tmp, &map->sid);
+       make_a_mod(mods, ldap_op, "ntSid", tmp);
+
+       snprintf(tmp, sizeof(tmp)-1, "%i", map->sid_name_use);
+       make_a_mod(mods, ldap_op, "ntGroupType", tmp);
+
+       make_a_mod(mods, ldap_op, "displayName", map->nt_name);
+       make_a_mod(mods, ldap_op, "description", map->comment);
+
+       return True;
+}
+
+static NTSTATUS ldapsam_getgroup(struct pdb_methods *methods,
+                                const char *filter,
+                                GROUP_MAP *map)
+{
+       struct ldapsam_privates *ldap_state =
+               (struct ldapsam_privates *)methods->private_data;
+       LDAPMessage *result;
+       LDAPMessage *entry;
+       int count;
+
+       if (ldapsam_search_one_group(ldap_state, filter, &result)
+           != LDAP_SUCCESS) {
+               return NT_STATUS_NO_SUCH_GROUP;
+       }
+
+       count = ldap_count_entries(ldap_state->ldap_struct, result);
+
+       if (count < 1) {
+               DEBUG(4, ("Did not find group for filter %s\n", filter));
+               return NT_STATUS_NO_SUCH_GROUP;
+       }
+
+       if (count > 1) {
+               DEBUG(1, ("Duplicate entries for filter %s: count=%d\n",
+                         filter, count));
+               return NT_STATUS_NO_SUCH_GROUP;
+       }
+
+       entry = ldap_first_entry(ldap_state->ldap_struct, result);
+
+       if (!entry) {
+               ldap_msgfree(result);
+               return NT_STATUS_UNSUCCESSFUL;
+       }
+
+       if (!init_group_from_ldap(ldap_state, map, entry)) {
+               DEBUG(1, ("init_group_from_ldap failed for group filter %s\n",
+                         filter));
+               ldap_msgfree(result);
+               return NT_STATUS_NO_SUCH_GROUP;
+       }
+
+       ldap_msgfree(result);
+       return NT_STATUS_OK;
+}
+
+static NTSTATUS ldapsam_getgrsid(struct pdb_methods *methods, GROUP_MAP *map,
+                                DOM_SID sid, BOOL with_priv)
+{
+       pstring filter;
+
+       snprintf(filter, sizeof(filter)-1,
+                "(&(objectClass=sambaGroupMapping)(ntSid=%s))",
+                sid_string_static(&sid));
+
+       return ldapsam_getgroup(methods, filter, map);
+}
+
+static NTSTATUS ldapsam_getgrgid(struct pdb_methods *methods, GROUP_MAP *map,
+                                gid_t gid, BOOL with_priv)
+{
+       pstring filter;
+
+       snprintf(filter, sizeof(filter)-1,
+                "(&(objectClass=sambaGroupMapping)(gidNumber=%d))",
+                gid);
+
+       return ldapsam_getgroup(methods, filter, map);
+}
+
+static NTSTATUS ldapsam_getgrnam(struct pdb_methods *methods, GROUP_MAP *map,
+                                char *name, BOOL with_priv)
+{
+       pstring filter;
+
+       /* TODO: Escaping of name? */
+
+       snprintf(filter, sizeof(filter)-1,
+                "(&(objectClass=sambaGroupMapping)(displayName=%s))",
+                name);
+
+       return ldapsam_getgroup(methods, filter, map);
+}
+
+static int ldapsam_search_one_group_by_gid(struct ldapsam_privates *ldap_state,
+                                          gid_t gid,
+                                          LDAPMessage **result)
+{
+       pstring filter;
+
+       snprintf(filter, sizeof(filter)-1,
+                "(&(objectClass=posixGroup)(gidNumber=%i))", gid);
+
+       return ldapsam_search_one_group(ldap_state, filter, result);
+}
+
+static NTSTATUS ldapsam_add_group_mapping_entry(struct pdb_methods *methods,
+                                               GROUP_MAP *map)
+{
+       struct ldapsam_privates *ldap_state =
+               (struct ldapsam_privates *)methods->private_data;
+       LDAPMessage *result = NULL;
+       LDAPMod **mods = NULL;
+
+       char *tmp;
+       pstring dn;
+       LDAPMessage *entry;
+
+       GROUP_MAP dummy;
+
+       int rc;
+
+       if (NT_STATUS_IS_OK(ldapsam_getgrgid(methods, &dummy,
+                                            map->gid, False))) {
+               DEBUG(0, ("Group %i already exists in LDAP\n", map->gid));
+               return NT_STATUS_UNSUCCESSFUL;
+       }
+
+       rc = ldapsam_search_one_group_by_gid(ldap_state, map->gid, &result);
+       if (rc != LDAP_SUCCESS) {
+               return NT_STATUS_UNSUCCESSFUL;
+       }
+
+       if (ldap_count_entries(ldap_state->ldap_struct, result) != 1) {
+               DEBUG(2, ("Group %i must exist exactly once in LDAP\n",
+                         map->gid));
+               ldap_msgfree(result);
+               return NT_STATUS_UNSUCCESSFUL;
+       }
+
+       entry = ldap_first_entry(ldap_state->ldap_struct, result);
+       tmp = ldap_get_dn(ldap_state->ldap_struct, entry);
+       pstrcpy(dn, tmp);
+       ldap_memfree(tmp);
+       ldap_msgfree(result);
+
+       if (!init_ldap_from_group(ldap_state, &mods, LDAP_MOD_ADD, map)) {
+               DEBUG(0, ("init_ldap_from_group failed!\n"));
+               ldap_mods_free(mods, 1);
+               return NT_STATUS_UNSUCCESSFUL;
+       }
+
+       if (mods == NULL) {
+               DEBUG(0, ("mods is empty\n"));
+               return NT_STATUS_UNSUCCESSFUL;
+       }
+
+       make_a_mod(&mods, LDAP_MOD_ADD, "objectClass",
+                  "sambaGroupMapping");
+
+       rc = ldapsam_modify(ldap_state, dn, mods);
+       ldap_mods_free(mods, 1);
+
+       if (rc != LDAP_SUCCESS) {
+               DEBUG(0, ("failed to modify group %i\n", map->gid));
+               return NT_STATUS_UNSUCCESSFUL;
+       }
+
+       DEBUG(2, ("successfully modified group %i in LDAP\n", map->gid));
+       return NT_STATUS_OK;
+}
+
+static NTSTATUS ldapsam_update_group_mapping_entry(struct pdb_methods *methods,
+                                                  GROUP_MAP *map)
+{
+       struct ldapsam_privates *ldap_state =
+               (struct ldapsam_privates *)methods->private_data;
+       int rc;
+       char *dn;
+       LDAPMessage *result;
+       LDAPMessage *entry;
+       LDAPMod **mods;
+
+       if (!init_ldap_from_group(ldap_state, &mods, LDAP_MOD_REPLACE, map)) {
+               DEBUG(0, ("init_ldap_from_group failed\n"));
+               return NT_STATUS_UNSUCCESSFUL;
+       }
+
+       if (mods == NULL) {
+               DEBUG(4, ("mods is empty: nothing to do\n"));
+               return NT_STATUS_UNSUCCESSFUL;
+       }
+
+       rc = ldapsam_search_one_group_by_gid(ldap_state, map->gid, &result);
+
+       if (rc != LDAP_SUCCESS) {
+               ldap_mods_free(mods, 1);
+               return NT_STATUS_UNSUCCESSFUL;
+       }
+
+       if (ldap_count_entries(ldap_state->ldap_struct, result) == 0) {
+               DEBUG(0, ("No group to modify!\n"));
+               ldap_msgfree(result);
+               ldap_mods_free(mods, 1);
+               return NT_STATUS_UNSUCCESSFUL;
+       }
+
+       entry = ldap_first_entry(ldap_state->ldap_struct, result);
+       dn = ldap_get_dn(ldap_state->ldap_struct, entry);
+        ldap_msgfree(result);
+
+       rc = ldapsam_modify(ldap_state, dn, mods);
+
+       ldap_mods_free(mods, 1);
+
+       if (rc != LDAP_SUCCESS) {
+               DEBUG(0, ("failed to modify group %i\n", map->gid));
+       }
+
+       DEBUG(2, ("successfully modified group %i in LDAP\n", map->gid));
+       return NT_STATUS_OK;
+}
+
+static NTSTATUS ldapsam_delete_group_mapping_entry(struct pdb_methods *methods,
+                                                  DOM_SID sid)
+{
+       struct ldapsam_privates *ldap_state =
+               (struct ldapsam_privates *)methods->private_data;
+       pstring sidstring, filter;
+       LDAPMessage *result;
+       int rc;
+       NTSTATUS ret;
+
+       const char *sam_group_attrs[] = { "ntSid", "ntGroupType",
+                                         "description", "displayName",
+                                         NULL };
+       sid_to_string(sidstring, &sid);
+       snprintf(filter, sizeof(filter)-1,
+                "(&(objectClass=sambaGroupMapping)(ntSid=%s))", sidstring);
+
+       rc = ldapsam_search_one_group(ldap_state, filter, &result);
+
+       if (rc != LDAP_SUCCESS) {
+               return NT_STATUS_NO_SUCH_GROUP;
+       }
+
+       ret = ldapsam_delete_entry(ldap_state, result, "sambaGroupMapping",
+                                  sam_group_attrs);
+       ldap_msgfree(result);
+       return ret;
+}
+
+static NTSTATUS ldapsam_setsamgrent(struct pdb_methods *my_methods,
+                                   BOOL update)
+{
+       struct ldapsam_privates *ldap_state =
+               (struct ldapsam_privates *)my_methods->private_data;
+       const char *filter = "(objectClass=sambaGroupMapping)";
+       int rc;
+
+       rc = ldapsam_search(ldap_state, lp_ldap_suffix(),
+                           LDAP_SCOPE_SUBTREE, filter,
+                           group_attr, 0, &ldap_state->result);
+
+       if (rc != LDAP_SUCCESS) {
+               DEBUG(0, ("LDAP search failed: %s\n", ldap_err2string(rc)));
+               DEBUG(3, ("Query was: %s, %s\n", lp_ldap_suffix(), filter));
+               ldap_msgfree(ldap_state->result);
+               ldap_state->result = NULL;
+               return NT_STATUS_UNSUCCESSFUL;
+       }
+
+       DEBUG(2, ("ldapsam_setsampwent: %d entries in the base!\n",
+                 ldap_count_entries(ldap_state->ldap_struct,
+                                    ldap_state->result)));
+
+       ldap_state->entry = ldap_first_entry(ldap_state->ldap_struct,
+                                ldap_state->result);
+       ldap_state->index = 0;
+
+       return NT_STATUS_OK;
+}
+
+static void ldapsam_endsamgrent(struct pdb_methods *my_methods)
+{
+       ldapsam_endsampwent(my_methods);
+}
+
+static NTSTATUS ldapsam_getsamgrent(struct pdb_methods *my_methods,
+                                   GROUP_MAP *map)
+{
+       NTSTATUS ret = NT_STATUS_UNSUCCESSFUL;
+       struct ldapsam_privates *ldap_state = (struct ldapsam_privates *)my_methods->private_data;
+       BOOL bret = False;
+
+       /* The rebind proc needs this *HACK*.  We are not multithreaded, so
+          this will work, but it's not nice. */
+       static_ldap_state = ldap_state;
+
+       while (!bret) {
+               if (!ldap_state->entry)
+                       return ret;
+               
+               ldap_state->index++;
+               bret = init_group_from_ldap(ldap_state, map, ldap_state->entry);
+               
+               ldap_state->entry = ldap_next_entry(ldap_state->ldap_struct,
+                                           ldap_state->entry); 
+       }
+
+       return NT_STATUS_OK;
+}
+
+static NTSTATUS ldapsam_enum_group_mapping(struct pdb_methods *methods,
+                                          enum SID_NAME_USE sid_name_use,
+                                          GROUP_MAP **rmap, int *num_entries,
+                                          BOOL unix_only, BOOL with_priv)
+{
+       GROUP_MAP map;
+       GROUP_MAP *mapt;
+       int entries = 0;
+       NTSTATUS nt_status;
+
+       *num_entries = 0;
+       *rmap = NULL;
+
+       if (!NT_STATUS_IS_OK(ldapsam_setsamgrent(methods, False))) {
+               DEBUG(0, ("Unable to open passdb\n"));
+               return NT_STATUS_ACCESS_DENIED;
+       }
+
+       while (NT_STATUS_IS_OK(nt_status = ldapsam_getsamgrent(methods, &map))) {
+               if (sid_name_use != SID_NAME_UNKNOWN &&
+                   sid_name_use != map.sid_name_use) {
+                       DEBUG(11,("enum_group_mapping: group %s is not of the requested type\n", map.nt_name));
+                       continue;
+               }
+               if (unix_only==ENUM_ONLY_MAPPED && map.gid==-1) {
+                       DEBUG(11,("enum_group_mapping: group %s is non mapped\n", map.nt_name));
+                       continue;
+               }
+
+               mapt=(GROUP_MAP *)Realloc((*rmap), (entries+1)*sizeof(GROUP_MAP));
+               if (!mapt) {
+                       DEBUG(0,("enum_group_mapping: Unable to enlarge group map!\n"));
+                       SAFE_FREE(*rmap);
+                       return NT_STATUS_UNSUCCESSFUL;
+               }
+               else
+                       (*rmap) = mapt;
+
+               mapt[entries] = map;
+
+               entries += 1;
+
+       }
+       ldapsam_endsamgrent(methods);
+
+       *num_entries = entries;
+
+       return NT_STATUS_OK;
+}
+
 NTSTATUS pdb_init_ldapsam(PDB_CONTEXT *pdb_context, PDB_METHODS **pdb_method, const char *location)
 {
        NTSTATUS nt_status;
@@ -1555,6 +2538,14 @@ NTSTATUS pdb_init_ldapsam(PDB_CONTEXT *pdb_context, PDB_METHODS **pdb_method, co
        (*pdb_method)->update_sam_account = ldapsam_update_sam_account;
        (*pdb_method)->delete_sam_account = ldapsam_delete_sam_account;
 
+       (*pdb_method)->getgrsid = ldapsam_getgrsid;
+       (*pdb_method)->getgrgid = ldapsam_getgrgid;
+       (*pdb_method)->getgrnam = ldapsam_getgrnam;
+       (*pdb_method)->add_group_mapping_entry = ldapsam_add_group_mapping_entry;
+       (*pdb_method)->update_group_mapping_entry = ldapsam_update_group_mapping_entry;
+       (*pdb_method)->delete_group_mapping_entry = ldapsam_delete_group_mapping_entry;
+       (*pdb_method)->enum_group_mapping = ldapsam_enum_group_mapping;
+
        /* TODO: Setup private data and free */
 
        ldap_state = talloc_zero(pdb_context->mem_ctx, sizeof(struct ldapsam_privates));
@@ -1566,8 +2557,23 @@ NTSTATUS pdb_init_ldapsam(PDB_CONTEXT *pdb_context, PDB_METHODS **pdb_method, co
 
        if (location) {
                ldap_state->uri = talloc_strdup(pdb_context->mem_ctx, location);
+#ifdef WITH_LDAP_SAMCONFIG
+       } else {
+               int ldap_port = lp_ldap_port();
+                       
+               /* remap default port if not using SSL (ie clear or TLS) */
+               if ( (lp_ldap_ssl() != LDAP_SSL_ON) && (ldap_port == 636) ) {
+                       ldap_port = 389;
+               }
+
+               ldap_state->uri = talloc_asprintf(pdb_context->mem_ctx, "%s://%s:%d", lp_ldap_ssl() == LDAP_SSL_ON ? "ldaps" : "ldap", lp_ldap_server(), ldap_port);
+               if (!ldap_state->uri) {
+                       return NT_STATUS_NO_MEMORY;
+               }
+#else
        } else {
                ldap_state->uri = "ldap://localhost";
+#endif
        }
 
        (*pdb_method)->private_data = ldap_state;