cli_setup_creds new arguments added.
[samba.git] / source3 / smbd / password.c
index b69b58f54e09520c3b8537c79356d551e96831e0..e1a4e66c4093eaf613d8a019c8749d87e0780cb0 100644 (file)
 
 #include "includes.h"
 
-#if (defined(NETGROUP) && defined (AUTOMOUNT))
-#include "rpcsvc/ypclnt.h"
-#endif
-
 extern int DEBUGLEVEL;
 extern int Protocol;
 
 /* users from session setup */
 static pstring session_users="";
 
-/* these are kept here to keep the string_combinations function simple */
-static char this_user[100]="";
-static char this_salt[100]="";
-static char this_crypted[100]="";
+extern pstring scope;
+extern pstring global_myname;
+extern fstring global_myworkgroup;
 
 /* Data to do lanman1/2 password challenge. */
 static unsigned char saved_challenge[8];
@@ -78,7 +73,7 @@ void generate_next_challenge(char *challenge)
 /*******************************************************************
 set the last challenge sent, usually from a password server
 ********************************************************************/
-BOOL set_challenge(char *challenge)
+BOOL set_challenge(unsigned char *challenge)
 {
   memcpy(saved_challenge,challenge,8);
   challenge_sent = True;
@@ -88,7 +83,7 @@ BOOL set_challenge(char *challenge)
 /*******************************************************************
 get the last challenge sent
 ********************************************************************/
-BOOL last_challenge(char *challenge)
+static BOOL last_challenge(unsigned char *challenge)
 {
   if (!challenge_sent) return(False);
   memcpy(challenge,saved_challenge,8);
@@ -110,7 +105,7 @@ user_struct *get_valid_user_struct(uint16 vuid)
     return NULL;
   vuid -= VUID_OFFSET;
   if ((vuid >= (uint16)num_validated_users) || 
-     (validated_users[vuid].uid == -1) || (validated_users[vuid].gid == -1))
+     (validated_users[vuid].uid == (uid_t)-1) || (validated_users[vuid].gid == (gid_t)-1))
     return NULL;
   return &validated_users[vuid];
 }
@@ -124,24 +119,21 @@ void invalidate_vuid(uint16 vuid)
 
   if (vuser == NULL) return;
 
-  vuser->uid = -1;
-  vuser->gid = -1;
+  vuser->uid = (uid_t)-1;
+  vuser->gid = (gid_t)-1;
 
   vuser->n_sids = 0;
 
-  /* same number of igroups as groups as attrs */
+  /* same number of igroups as groups */
   vuser->n_groups = 0;
 
-  if (vuser->groups && (vuser->groups != (gid_t *)vuser->igroups))
-       free(vuser->groups);
+  if (vuser->groups)
+    free((char *)vuser->groups);
 
-  if (vuser->igroups) free(vuser->igroups);
-  if (vuser->attrs  ) free(vuser->attrs);
-  if (vuser->sids   ) free(vuser->sids);
+  if (vuser->sids)
+    free((char *)vuser->sids);
 
-  vuser->attrs   = NULL;
   vuser->sids    = NULL;
-  vuser->igroups = NULL;
   vuser->groups  = NULL;
 }
 
@@ -158,97 +150,13 @@ char *validated_username(uint16 vuid)
 }
 
 
-/****************************************************************************
-Setup the groups a user belongs to.
-****************************************************************************/
-int setup_groups(char *user, int uid, int gid, int *p_ngroups, 
-                int **p_igroups, gid_t **p_groups,
-         int **p_attrs)
-{
-  if (-1 == initgroups(user,gid))
-    {
-      if (getuid() == 0)
-       {
-         DEBUG(0,("Unable to initgroups!\n"));
-         if (gid < 0 || gid > 16000 || uid < 0 || uid > 16000)
-           DEBUG(0,("This is probably a problem with the account %s\n",user));
-       }
-    }
-  else
-    {
-      int i,ngroups;
-      int *igroups;
-      int *attrs;
-      gid_t grp = 0;
-      ngroups = getgroups(0,&grp);
-      if (ngroups <= 0)
-        ngroups = 32;
-      igroups = (int *)malloc(sizeof(int)*ngroups);
-      attrs   = (int *)malloc(sizeof(int)*ngroups);
-      for (i=0;i<ngroups;i++)
-      {
-        attrs  [i] = 0x7; /* XXXX don't know what NT user attributes are yet! */
-        igroups[i] = 0x42424242;
-      }
-      ngroups = getgroups(ngroups,(gid_t *)igroups);
-
-      if (igroups[0] == 0x42424242)
-        ngroups = 0;
-
-      *p_ngroups = ngroups;
-      *p_attrs   = attrs;
-
-      /* The following bit of code is very strange. It is due to the
-         fact that some OSes use int* and some use gid_t* for
-         getgroups, and some (like SunOS) use both, one in prototypes,
-         and one in man pages and the actual code. Thus we detect it
-         dynamically using some very ugly code */
-      if (ngroups > 0)
-        {
-         /* does getgroups return ints or gid_t ?? */
-         static BOOL groups_use_ints = True;
-
-         if (groups_use_ints && 
-             ngroups == 1 && 
-             SVAL(igroups,2) == 0x4242)
-           groups_use_ints = False;
-         
-          for (i=0;groups_use_ints && i<ngroups;i++)
-            if (igroups[i] == 0x42424242)
-             groups_use_ints = False;
-             
-          if (groups_use_ints)
-          {
-             *p_igroups = igroups;
-             *p_groups = (gid_t *)igroups;       
-          }
-          else
-          {
-             gid_t *groups = (gid_t *)igroups;
-             igroups = (int *)malloc(sizeof(int)*ngroups);
-             for (i=0;i<ngroups;i++)
-          {
-               igroups[i] = groups[i];
-          }
-             *p_igroups = igroups;
-             *p_groups = (gid_t *)groups;
-           }
-       }
-      DEBUG(3,("%s is in %d groups\n",user,ngroups));
-      for (i=0;i<ngroups;i++)
-        DEBUG(3,("%d ",igroups[i]));
-      DEBUG(3,("\n"));
-    }
-  return 0;
-}
-
 
 /****************************************************************************
 register a uid/name pair as being valid and that a valid password
 has been given. vuid is biased by an offset. This allows us to
 tell random client vuid's (normally zero) from valid vuids.
 ****************************************************************************/
-uint16 register_vuid(int uid,int gid, char *name,BOOL guest)
+uint16 register_vuid(uid_t uid,gid_t gid, char *unix_name, char *requested_name, BOOL guest)
 {
   user_struct *vuser;
   struct passwd *pwfile; /* for getting real name from passwd file */
@@ -295,25 +203,22 @@ uint16 register_vuid(int uid,int gid, char *name,BOOL guest)
   vuser->uid = uid;
   vuser->gid = gid;
   vuser->guest = guest;
-  strcpy(vuser->name,name);
+  fstrcpy(vuser->name,unix_name);
+  fstrcpy(vuser->requested_name,requested_name);
 
   vuser->n_sids = 0;
   vuser->sids   = NULL;
 
   vuser->n_groups = 0;
   vuser->groups  = NULL;
-  vuser->igroups = NULL;
-  vuser->attrs    = NULL;
 
   /* Find all the groups this uid is in and store them. 
      Used by become_user() */
-  setup_groups(name,uid,gid,
+  get_unixgroups(unix_name,uid,gid,
               &vuser->n_groups,
-              &vuser->igroups,
-              &vuser->groups,
-              &vuser->attrs);
+              &vuser->groups);
 
-  DEBUG(3,("uid %d registered to name %s\n",uid,name));
+  DEBUG(3,("uid %d registered to name %s\n",(int)uid,unix_name));
 
   DEBUG(3, ("Clearing default real name\n"));
   fstrcpy(vuser->real_name, "<Full Name>\0");
@@ -345,594 +250,43 @@ void add_session_user(char *user)
        DEBUG(1,("Too many session users??\n"));
       else
        {
-         strcat(session_users," ");
-         strcat(session_users,suser);
+         pstrcat(session_users," ");
+         pstrcat(session_users,suser);
        }
     }
 }
 
 
-#ifdef NO_GETSPNAM
-/* a fake shadow password routine which just fills a fake spwd struct
- * with the sp_pwdp field. (sreiz@aie.nl)
- */
-static struct spwd *getspnam(char *username) /* fake shadow password routine */
-{
-       FILE *f;
-       char line[1024];
-       static char pw[20];
-       static struct spwd static_spwd;
-
-       static_spwd.sp_pwdp=0;
-       if (!(f=fopen("/etc/master.passwd", "r")))
-               return 0;
-       while (fgets(line, 1024, f)) {
-               if (!strncmp(line, username, strlen(username)) &&
-                line[strlen(username)]==':') { /* found entry */
-                       char *p, *q;
-
-                       p=line+strlen(username)+1;
-                       if ((q=strchr(p, ':'))) {
-                               *q=0;
-                               if (q-p+1>20)
-                                       break;
-                               strcpy(pw, p);
-                               static_spwd.sp_pwdp=pw;
-                       }
-                       break;
-               }
-       }
-       fclose(f);
-       if (static_spwd.sp_pwdp)
-               return &static_spwd;
-       return 0;
-}
-#endif
-
-
-#ifdef OSF1_ENH_SEC
 /****************************************************************************
-an enhanced crypt for OSF1
-****************************************************************************/
-static char *osf1_bigcrypt(char *password,char *salt1)
+update the encrypted smbpasswd file from the plaintext username and password
+*****************************************************************************/
+static BOOL update_smbpassword_file(char *user, char *password)
 {
-  static char result[AUTH_MAX_PASSWD_LENGTH] = "";
-  char *p1;
-  char *p2=password;
-  char salt[3];
-  int i;
-  int parts = strlen(password) / AUTH_CLEARTEXT_SEG_CHARS;
-  if (strlen(password)%AUTH_CLEARTEXT_SEG_CHARS)
-    parts++;
-
-  StrnCpy(salt,salt1,2);
-  StrnCpy(result,salt1,2);
-
-  for (i=0; i<parts;i++)
-    {
-      p1 = crypt(p2,salt);
-      strcat(result,p1+2);
-      StrnCpy(salt,&result[2+i*AUTH_CIPHERTEXT_SEG_CHARS],2);
-      p2 += AUTH_CLEARTEXT_SEG_CHARS;
-    }
-
-  return(result);
-}
-#endif
-
-
-/****************************************************************************
-update the enhanced security database. Only relevant for OSF1 at the moment.
-****************************************************************************/
-static void update_protected_database( char *user, BOOL result)
-{
-#ifdef OSF1_ENH_SEC
-  struct pr_passwd *mypasswd;
-  time_t starttime;
-
-  mypasswd = getprpwnam (user);
-  starttime = time (NULL);
-
-  if (result)
-    {
-      mypasswd->ufld.fd_slogin = starttime;
-      mypasswd->ufld.fd_nlogins = 0;
-      
-      putprpwnam(user,mypasswd);
-      
-      DEBUG(3,("Update protected database for Account %s after succesful connection\n",user));
-    }
-  else
-    {
-      mypasswd->ufld.fd_ulogin = starttime;
-      mypasswd->ufld.fd_nlogins = mypasswd->ufld.fd_nlogins + 1;
-      if ( mypasswd->ufld.fd_max_tries != 0 && mypasswd->ufld.fd_nlogins > mypasswd->ufld.fd_max_tries )
-       {
-         mypasswd->uflg.fg_lock = 0;
-         DEBUG(3,("Account is disabled -- see Account Administrator.\n"));
-       }
-      putprpwnam ( user , mypasswd );
-      DEBUG(3,("Update protected database for Account %s after refusing connection\n",user));
-    }
-#else
-  DEBUG(6,("Updated database with %s %s\n",user,BOOLSTR(result)));
-#endif
-}
-
-
-#ifdef USE_PAM
-/*******************************************************************
-check on PAM authentication
-********************************************************************/
-
-/* We first need some helper functions */
-#include <security/pam_appl.h>
-/* Static variables used to communicate between the conversation function
- * and the server_login function
- */
-static char *PAM_username;
-static char *PAM_password;
-
-/* PAM conversation function
- * Here we assume (for now, at least) that echo on means login name, and
- * echo off means password.
- */
-static int PAM_conv (int num_msg,
-                     const struct pam_message **msg,
-                     struct pam_response **resp,
-                     void *appdata_ptr) {
-  int replies = 0;
-  struct pam_response *reply = NULL;
-
-  #define COPY_STRING(s) (s) ? strdup(s) : NULL
-
-  reply = malloc(sizeof(struct pam_response) * num_msg);
-  if (!reply) return PAM_CONV_ERR;
-
-  for (replies = 0; replies < num_msg; replies++) {
-    switch (msg[replies]->msg_style) {
-      case PAM_PROMPT_ECHO_ON:
-        reply[replies].resp_retcode = PAM_SUCCESS;
-        reply[replies].resp = COPY_STRING(PAM_username);
-          /* PAM frees resp */
-        break;
-      case PAM_PROMPT_ECHO_OFF:
-        reply[replies].resp_retcode = PAM_SUCCESS;
-        reply[replies].resp = COPY_STRING(PAM_password);
-          /* PAM frees resp */
-        break;
-      case PAM_TEXT_INFO:
-       /* fall through */
-      case PAM_ERROR_MSG:
-        /* ignore it... */
-        reply[replies].resp_retcode = PAM_SUCCESS;
-        reply[replies].resp = NULL;
-        break;
-      default:
-        /* Must be an error of some sort... */
-        free (reply);
-        return PAM_CONV_ERR;
-    }
-  }
-  if (reply) *resp = reply;
-  return PAM_SUCCESS;
-}
-static struct pam_conv PAM_conversation = {
-    &PAM_conv,
-    NULL
-};
-
-
-static BOOL pam_auth(char *this_user,char *password)
-{
-  pam_handle_t *pamh;
-  int pam_error;
-
-  /* Now use PAM to do authentication.  For now, we won't worry about
-   * session logging, only authentication.  Bail out if there are any
-   * errors.  Since this is a limited protocol, and an even more limited
-   * function within a server speaking this protocol, we can't be as
-   * verbose as would otherwise make sense.
-   * Query: should we be using PAM_SILENT to shut PAM up?
-   */
-  #define PAM_BAIL if (pam_error != PAM_SUCCESS) { \
-     pam_end(pamh, 0); return False; \
-   }
-  PAM_password = password;
-  PAM_username = this_user;
-  pam_error = pam_start("samba", this_user, &PAM_conversation, &pamh);
-  PAM_BAIL;
-/* Setting PAM_SILENT stops generation of error messages to syslog
- * to enable debugging on Red Hat Linux set:
- * /etc/pam.d/samba:
- *     auth required /lib/security/pam_pwdb.so nullok shadow audit
- * _OR_ change PAM_SILENT to 0 to force detailed reporting (logging)
- */
-  pam_error = pam_authenticate(pamh, PAM_SILENT);
-  PAM_BAIL;
-  /* It is not clear to me that account management is the right thing
-   * to do, but it is not clear that it isn't, either.  This can be
-   * removed if no account management should be done.  Alternately,
-   * put a pam_allow.so entry in /etc/pam.conf for account handling. */
-  pam_error = pam_acct_mgmt(pamh, PAM_SILENT);
-  PAM_BAIL;
-  pam_end(pamh, PAM_SUCCESS);
-  /* If this point is reached, the user has been authenticated. */
-  return(True);
-}
-#endif
-
-
-#ifdef AFS_AUTH
-/*******************************************************************
-check on AFS authentication
-********************************************************************/
-static BOOL afs_auth(char *this_user,char *password)
-{
-  long password_expires = 0;
-  char *reason;
-    
-  /* For versions of AFS prior to 3.3, this routine has few arguments, */
-  /* but since I can't find the old documentation... :-)               */
-  setpag();
-  if (ka_UserAuthenticateGeneral(KA_USERAUTH_VERSION+KA_USERAUTH_DOSETPAG,
-                                this_user,
-                                (char *) 0, /* instance */
-                                (char *) 0, /* cell */
-                                password,
-                                0,          /* lifetime, default */
-                                &password_expires, /*days 'til it expires */
-                                0,          /* spare 2 */
-                                &reason) == 0)
-    return(True);
-  return(False);
-}
-#endif
-
-
-#ifdef DFS_AUTH
-
-sec_login_handle_t my_dce_sec_context;
-int dcelogin_atmost_once = 0;
-
-/*******************************************************************
-check on a DCE/DFS authentication
-********************************************************************/
-static BOOL dfs_auth(char *this_user,char *password)
-{
-  error_status_t err;
-  int err2;
-  int prterr;
-  boolean32 password_reset;
-  sec_passwd_rec_t my_dce_password;
-  sec_login_auth_src_t auth_src = sec_login_auth_src_network;
-  unsigned char dce_errstr[dce_c_error_string_len];
-
-  /*
-   * We only go for a DCE login context if the given password
-   * matches that stored in the local password file.. 
-   * Assumes local passwd file is kept in sync w/ DCE RGY!
-   */
-
-  /* Fix for original (broken) code from Brett Wooldridge <brettw@austin.ibm.com> */
-  if (dcelogin_atmost_once)
-    return (False);
-  /* This can be ifdefed as the DCE check below is stricter... */
-#ifndef NO_CRYPT
-  if ( strcmp((char *)crypt(password,this_salt),this_crypted) )
-    return (False);
-#endif
-
-  if (sec_login_setup_identity(
-                              (unsigned char *)this_user,
-                              sec_login_no_flags,
-                              &my_dce_sec_context,
-                              &err) == 0)
-    {
-      dce_error_inq_text(err, dce_errstr, &err2);
-      DEBUG(0,("DCE Setup Identity for %s failed: %s\n",
-              this_user,dce_errstr));
-      return(False);
-    }
-
-  my_dce_password.version_number = sec_passwd_c_version_none;
-  my_dce_password.pepper = NULL; 
-  my_dce_password.key.key_type = sec_passwd_plain;
-  my_dce_password.key.tagged_union.plain  = (idl_char *)password;
-  
-  if (sec_login_valid_and_cert_ident(my_dce_sec_context,
-                                    &my_dce_password,
-                                    &password_reset,
-                                    &auth_src,
-                                    &err) == 0 )
-    { 
-      dce_error_inq_text(err, dce_errstr, &err2);
-      DEBUG(0,("DCE Identity Validation failed for principal %s: %s\n",
-              this_user,dce_errstr));
-         
-      return(False);
-    }
-
-  sec_login_set_context(my_dce_sec_context, &err);
-  if (err != error_status_ok )
-    {  
-      dce_error_inq_text(err, dce_errstr, &err2);
-      DEBUG(0,("DCE login failed for principal %s, cant set context: %s\n",
-              this_user,dce_errstr));
-      sec_login_purge_context(my_dce_sec_context, &err);
-      return(False);
-    }
-  else
-    {
-      DEBUG(0,("DCE login succeeded for principal %s on pid %d\n",
-              this_user, getpid()));
-    }
-
-  dcelogin_atmost_once = 1;
-  return (True);
-}
-
-void dfs_unlogin(void)
-{
-  error_status_t err;
-  int err2;
-  unsigned char dce_errstr[dce_c_error_string_len];
-
-  sec_login_purge_context(my_dce_sec_context, &err);
-  if (err != error_status_ok )
-    {  
-      dce_error_inq_text(err, dce_errstr, &err2);
-      DEBUG(0,("DCE purge login context failed for server instance %d: %s\n",
-              getpid(), dce_errstr));
-    }
-}
-
-#endif
-
-#ifdef KRB5_AUTH
-/*******************************************************************
-check on Kerberos authentication
-********************************************************************/
-static BOOL krb5_auth(char *this_user,char *password)
-{
-       krb5_data tgtname = {
-               0,
-               KRB5_TGS_NAME_SIZE,
-               KRB5_TGS_NAME
-       };
-       krb5_context kcontext;
-       krb5_principal kprinc;
-       krb5_principal server;
-       krb5_creds kcreds;
-       int options = 0;
-       krb5_address **addrs = (krb5_address **)0;
-       krb5_preauthtype *preauth = NULL;
-       krb5_keytab keytab = NULL;
-       krb5_timestamp now;
-       krb5_ccache ccache = NULL;
-       int retval;
-       char *name;
-
-       if ( retval=krb5_init_context(&kcontext))
-       {
-               return(False);
-       }
-
-       if ( retval = krb5_timeofday(kcontext, &now) )
-       {
-               return(False);
-       }
-
-       if ( retval = krb5_cc_default(kcontext, &ccache) )
-       {
-               return(False);
-       }
+       struct smb_passwd *smbpw;
+       BOOL ret;
        
-       if ( retval = krb5_parse_name(kcontext, this_user, &kprinc) )
-       {
-               return(False);
-       }
-
-       memset((char *)&kcreds, 0, sizeof(kcreds));
-
-       kcreds.client = kprinc;
+       become_root(0);
+       smbpw = getsmbpwnam(user);
+       unbecome_root(0);
        
-       if ((retval = krb5_build_principal_ext(kcontext, &server,
-               krb5_princ_realm(kcontext, kprinc)->length,
-               krb5_princ_realm(kcontext, kprinc)->data,
-               tgtname.length,
-               tgtname.data,
-               krb5_princ_realm(kcontext, kprinc)->length,
-               krb5_princ_realm(kcontext, kprinc)->data,
-               0)))
-       {
-               return(False);
+       if(smbpw == NULL) {
+               DEBUG(0,("getsmbpwnam returned NULL\n"));
+               return False;
        }
-
-       kcreds.server = server;
-
-       retval = krb5_get_in_tkt_with_password(kcontext,
-               options,
-               addrs,
-               NULL,
-               preauth,
-               password,
-               0,
-               &kcreds,
-               0);
-
-       if ( retval )
-       {
-               return(False);
+       /* Here, the flag is one, because we want to ignore the
+           XXXXXXX'd out password */
+       ret = change_oem_password( smbpw, password, True);
+       if (ret == False) {
+               DEBUG(3,("change_oem_password returned False\n"));
        }
 
-       return(True);
-}
-#endif /* KRB5_AUTH */
-
-#ifdef KRB4_AUTH
-/*******************************************************************
-check on Kerberos authentication
-********************************************************************/
-static BOOL krb4_auth(char *this_user,char *password)
-{
-  char realm[REALM_SZ];
-  char tkfile[MAXPATHLEN];
-  
-  if (krb_get_lrealm(realm, 1) != KSUCCESS)
-    (void) strncpy(realm, KRB_REALM, sizeof (realm));
-  
-  (void) sprintf(tkfile, "/tmp/samba_tkt_%d", getpid());
-  
-  krb_set_tkt_string(tkfile);
-  if (krb_verify_user(this_user, "", realm,
-                     password, 0,
-                     "rmcd") == KSUCCESS) {
-    unlink(tkfile);
-    return 1;
-  }
-  unlink(tkfile);
-  return 0;
-}
-#endif /* KRB4_AUTH */
-
-#ifdef LINUX_BIGCRYPT
-/****************************************************************************
-an enhanced crypt for Linux to handle password longer than 8 characters
-****************************************************************************/
-static int linux_bigcrypt(char *password,char *salt1, char *crypted)
-{
-#define LINUX_PASSWORD_SEG_CHARS 8
-  char salt[3];
-  int i;
-  
-  StrnCpy(salt,salt1,2);
-  crypted +=2;
-  
-  for ( i=strlen(password); i > 0; i -= LINUX_PASSWORD_SEG_CHARS) {
-    char * p = crypt(password,salt) + 2;
-    if (strncmp(p, crypted, LINUX_PASSWORD_SEG_CHARS) != 0)
-      return(0);
-    password += LINUX_PASSWORD_SEG_CHARS;
-    crypted  += strlen(p);
-  }
-  
-  return(1);
-}
-#endif
-
-
-/****************************************************************************
-apply a function to upper/lower case combinations
-of a string and return true if one of them returns true.
-try all combinations with N uppercase letters.
-offset is the first char to try and change (start with 0)
-it assumes the string starts lowercased
-****************************************************************************/
-static BOOL string_combinations2(char *s,int offset,BOOL (*fn)(char *),int N)
-{
-  int len = strlen(s);
-  int i;
-
-#ifdef PASSWORD_LENGTH
-  len = MIN(len,PASSWORD_LENGTH);
-#endif
-
-  if (N <= 0 || offset >= len)
-    return(fn(s));
-
-  for (i=offset;i<(len-(N-1));i++)
-    {      
-      char c = s[i];
-      if (!islower(c)) continue;
-      s[i] = toupper(c);
-      if (string_combinations2(s,i+1,fn,N-1))
-       return(True);
-      s[i] = c;
-    }
-  return(False);
-}
-
-/****************************************************************************
-apply a function to upper/lower case combinations
-of a string and return true if one of them returns true.
-try all combinations with up to N uppercase letters.
-offset is the first char to try and change (start with 0)
-it assumes the string starts lowercased
-****************************************************************************/
-static BOOL string_combinations(char *s,BOOL (*fn)(char *),int N)
-{
-  int n;
-  for (n=1;n<=N;n++)
-    if (string_combinations2(s,0,fn,n)) return(True);
-  return(False);
+       return ret;
 }
 
 
 
-/****************************************************************************
-core of password checking routine
-****************************************************************************/
-BOOL password_check(char *password)
-{
-
-#ifdef USE_PAM
-/* This falls through if the password check fails
-       - if NO_CRYPT is defined this causes an error msg
-               saying Warning - no crypt available
-       - if NO_CRYPT is NOT defined this is a potential security hole
-               as it may authenticate via the crypt call when PAM
-               settings say it should fail.
-  if (pam_auth(this_user,password)) return(True);
-Hence we make a direct return to avoid a second chance!!!
-*/
-  return (pam_auth(this_user,password));
-#endif
-
-#ifdef AFS_AUTH
-  if (afs_auth(this_user,password)) return(True);
-#endif
-
-#ifdef DFS_AUTH
-  if (dfs_auth(this_user,password)) return(True);
-#endif 
-
-#ifdef KRB5_AUTH
-  if (krb5_auth(this_user,password)) return(True);
-#endif
-
-#ifdef KRB4_AUTH
-  if (krb4_auth(this_user,password)) return(True);
-#endif
 
-#ifdef PWDAUTH
-  if (pwdauth(this_user,password) == 0)
-    return(True);
-#endif
-
-#ifdef OSF1_ENH_SEC
-  return(strcmp(osf1_bigcrypt(password,this_salt),this_crypted) == 0);
-#endif
-
-#ifdef ULTRIX_AUTH
-  return (strcmp((char *)crypt16(password, this_salt ),this_crypted) == 0);
-#endif
-
-#ifdef LINUX_BIGCRYPT
-  return(linux_bigcrypt(password,this_salt,this_crypted));
-#endif
-
-#ifdef HPUX_10_TRUSTED
-  return(bigcrypt(password,this_salt,this_crypted));
-#endif
-
-#ifdef NO_CRYPT
-  DEBUG(1,("Warning - no crypt available\n"));
-  return(False);
-#else
-  return(strcmp((char *)crypt(password,this_salt),this_crypted) == 0);
-#endif
-}
 
 /****************************************************************************
 core of smb password checking routine.
@@ -980,330 +334,200 @@ BOOL smb_password_check(char *password, unsigned char *part_passwd, unsigned cha
  Do a specific test for an smb password being correct, given a smb_password and
  the lanman and NT responses.
 ****************************************************************************/
-
-BOOL smb_password_ok(struct smb_passwd *smb_pass,
+BOOL smb_password_ok(struct smb_passwd *smb_pass, uchar chal[8],
                      uchar lm_pass[24], uchar nt_pass[24])
 {
-  uchar challenge[8];
+       uchar challenge[8];
 
-  if (!lm_pass || !smb_pass) return(False);
+       if (!lm_pass || !smb_pass) return(False);
 
-  if(smb_pass->acct_ctrl & ACB_DISABLED)
-  {
-    DEBUG(3,("smb_password_ok: account for user %s was disabled.\n", smb_pass->smb_name));
-    return(False);
-  }
+       DEBUG(4,("Checking SMB password for user %s\n", 
+                smb_pass->unix_name));
 
-  if (!last_challenge(challenge))
-  {
-    DEBUG(1,("smb_password_ok: no challenge done - password failed\n"));
-    return False;
-  }
+       if(smb_pass->acct_ctrl & ACB_DISABLED) {
+               DEBUG(3,("account for user %s was disabled.\n", 
+                        smb_pass->unix_name));
+               return(False);
+       }
 
-  DEBUG(4,("smb_password_ok: Checking SMB password for user %s\n", smb_pass->smb_name));
+       if (chal == NULL)
+       {
+               DEBUG(5,("use last SMBnegprot challenge\n"));
+               if (!last_challenge(challenge))
+               {
+                       DEBUG(1,("no challenge done - password failed\n"));
+                       return False;
+               }
+       }
+       else
+       {
+               DEBUG(5,("challenge received\n"));
+               memcpy(challenge, chal, 8);
+       }
 
-  if ((Protocol >= PROTOCOL_NT1) && (smb_pass->smb_nt_passwd != NULL))
-  {
-    /* We have the NT MD4 hash challenge available - see if we can
-       use it (ie. does it exist in the smbpasswd file).
-     */
-    DEBUG(4,("smb_password_ok: Checking NT MD4 password\n"));
-    if (smb_password_check(nt_pass, smb_pass->smb_nt_passwd, challenge))
-    {
-      DEBUG(4,("smb_password_ok: NT MD4 password check succeeded\n"));
-      return(True);
-    }
-    DEBUG(4,("smb_password_ok: NT MD4 password check failed\n"));
-  }
+       if ((Protocol >= PROTOCOL_NT1) && (smb_pass->smb_nt_passwd != NULL)) {
+               /* We have the NT MD4 hash challenge available - see if we can
+                  use it (ie. does it exist in the smbpasswd file).
+               */
+               DEBUG(4,("smb_password_ok: Checking NT MD4 password\n"));
+               if (smb_password_check((char *)nt_pass, 
+                                      (uchar *)smb_pass->smb_nt_passwd, 
+                                      challenge)) {
+                       DEBUG(4,("NT MD4 password check succeeded\n"));
+                       return(True);
+               }
+               DEBUG(4,("NT MD4 password check failed\n"));
+       }
 
-  /* Try against the lanman password. smb_pass->smb_passwd == NULL means
-     no password, allow access. */
+       /* Try against the lanman password. smb_pass->smb_passwd == NULL means
+          no password, allow access. */
 
-  DEBUG(4,("Checking LM MD4 password\n"));
+       DEBUG(4,("Checking LM MD4 password\n"));
 
-  if((smb_pass->smb_passwd == NULL) && (smb_pass->acct_ctrl & ACB_PWNOTREQ))
-  {
-    DEBUG(4,("smb_password_ok: no password required for user %s\n", smb_pass->smb_name));
-    return True;
-  }
+       if((smb_pass->smb_passwd == NULL) && 
+          (smb_pass->acct_ctrl & ACB_PWNOTREQ)) {
+               DEBUG(4,("no password required for user %s\n",
+                        smb_pass->unix_name));
+               return True;
+       }
 
-  if((smb_pass->smb_passwd != NULL) && smb_password_check(lm_pass, smb_pass->smb_passwd, challenge))
-  {
-    DEBUG(4,("smb_password_ok: LM MD4 password check succeeded\n"));
-    return(True);
-  }
+       if((smb_pass->smb_passwd != NULL) && 
+          smb_password_check((char *)lm_pass, 
+                             (uchar *)smb_pass->smb_passwd, challenge)) {
+               DEBUG(4,("LM MD4 password check succeeded\n"));
+               return(True);
+       }
 
-  DEBUG(4,("smb_password_ok: LM MD4 password check failed\n"));
+       DEBUG(4,("LM MD4 password check failed\n"));
 
-  return False;
+       return False;
 }
 
+
 /****************************************************************************
-check if a username/password is OK
+check if a username/password is OK assuming the password is a 24 byte
+SMB hash
+return True if the password is correct, False otherwise
 ****************************************************************************/
-BOOL password_ok(char *user,char *password, int pwlen, struct passwd *pwd)
-{
-  pstring pass2;
-  int level = lp_passwordlevel();
-  struct passwd *pass;
-  char challenge[8];
-  struct smb_passwd *smb_pass;
-  BOOL challenge_done = False;
-
-  if (password) password[pwlen] = 0;
 
-  if (pwlen == 24)
-    challenge_done = last_challenge(challenge);
-
-#if DEBUG_PASSWORD
-  if (challenge_done)
-    {
-      int i;      
-      DEBUG(100,("checking user=[%s] pass=[",user));
-      for( i = 0; i < 24; i++)
-       DEBUG(100,("%0x ", (unsigned char)password[i]));
-      DEBUG(100,("]\n"));
-    } else {
-           DEBUG(100,("checking user=[%s] pass=[%s]\n",user,password));
-    }
-#endif
-
-  if (!password)
-    return(False);
-
-  if (((!*password) || (!pwlen)) && !lp_null_passwords())
-    return(False);
-
-  if (pwd && !user) 
-    {
-      pass = (struct passwd *) pwd;
-      user = pass->pw_name;
-    } 
-  else 
-    pass = Get_Pwnam(user,True);
-
-  DEBUG(4,("SMB Password - pwlen = %d, challenge_done = %d\n", pwlen, challenge_done));
-
-  if ((pwlen == 24) && challenge_done)
-    {
-      DEBUG(4,("Checking SMB password for user %s (l=24)\n",user));
+BOOL pass_check_smb(char *user, char *domain,
+               uchar *chal, uchar *lm_pwd, uchar *nt_pwd,
+               struct passwd *pwd)
+{
+       struct passwd *pass;
+       struct smb_passwd *smb_pass;
 
-      if (!pass) 
+       if (!lm_pwd || !nt_pwd)
        {
-         DEBUG(3,("Couldn't find user %s\n",user));
-         return(False);
+               return(False);
        }
 
-      smb_pass = getsmbpwnam(user);
+       if (pwd != NULL && user == NULL)
+       {
+               pass = (struct passwd *) pwd;
+               user = pass->pw_name;
+       }
+       else
+       {
+               pass = Get_Pwnam(user,True);
+       }
 
-      if (!smb_pass)
+       if (pass == NULL)
        {
-         DEBUG(3,("Couldn't find user %s in smb_passwd file.\n", user));
-         return(False);
+               DEBUG(3,("Couldn't find user %s\n",user));
+               return(False);
        }
 
-      /* Quit if the account was disabled. */
-      if(smb_pass->acct_ctrl & ACB_DISABLED)
-        {
-          DEBUG(3,("password_ok: account for user %s was disabled.\n", user));
-          return(False);
-        }
+       smb_pass = getsmbpwnam(user);
 
-      /* Ensure the uid's match */
-      if (smb_pass->smb_userid != pass->pw_uid)
+       if (smb_pass == NULL)
        {
-         DEBUG(3,("Error : UNIX and SMB uids in password files do not match !\n"));
-         return(False);
+               DEBUG(3,("Couldn't find user %s in smb_passwd file.\n", user));
+               return(False);
        }
 
-      if(smb_password_ok( smb_pass, password, password))
-        {
-          update_protected_database(user,True);
-          return(True);
+       /* Quit if the account was disabled. */
+       if(smb_pass->acct_ctrl & ACB_DISABLED) {
+               DEBUG(3,("account for user %s was disabled.\n", user));
+               return(False);
         }
 
-      DEBUG(3,("Error smb_password_check failed\n"));
-    }
-
-  DEBUG(4,("Checking password for user %s (l=%d)\n",user,pwlen));
-
-  if (!pass) 
-    {
-      DEBUG(3,("Couldn't find user %s\n",user));
-      return(False);
-    }
-
-#ifdef SHADOW_PWD
-  {
-    struct spwd *spass;
-
-    /* many shadow systems require you to be root to get the password,
-       in most cases this should already be the case when this
-       function is called, except perhaps for IPC password changing
-       requests */
-
-    spass = getspnam(pass->pw_name);
-    if (spass && spass->sp_pwdp)
-      pass->pw_passwd = spass->sp_pwdp;
-  }
-#elif defined(IA_UINFO)
-  {
-      /* Need to get password with SVR4.2's ia_ functions instead of
-         get{sp,pw}ent functions. Required by UnixWare 2.x, tested on 
-         version 2.1. (tangent@cyberport.com) */
-      uinfo_t uinfo;
-      if (ia_openinfo(pass->pw_name, &uinfo) != -1)
-        ia_get_logpwd(uinfo, &(pass->pw_passwd));
-  }
-#endif
-
-#ifdef SecureWare
-  {
-    struct pr_passwd *pr_pw = getprpwnam(pass->pw_name);
-    if (pr_pw && pr_pw->ufld.fd_encrypt)
-      pass->pw_passwd = pr_pw->ufld.fd_encrypt;
-  }
-#endif
-
-#ifdef HPUX_10_TRUSTED
-  {
-    struct pr_passwd *pr_pw = getprpwnam(pass->pw_name);
-    if (pr_pw && pr_pw->ufld.fd_encrypt)
-      pass->pw_passwd = pr_pw->ufld.fd_encrypt;
-  }
-#endif
-
-#ifdef OSF1_ENH_SEC
-  {
-    struct pr_passwd *mypasswd;
-    DEBUG(5,("Checking password for user %s in OSF1_ENH_SEC\n",user));
-    mypasswd = getprpwnam (user);
-    if ( mypasswd )
-      { 
-       strcpy(pass->pw_name,mypasswd->ufld.fd_name);
-       strcpy(pass->pw_passwd,mypasswd->ufld.fd_encrypt);
-      }
-    else
-      {
-       DEBUG(5,("No entry for user %s in protected database !\n",user));
-       return(False);
-      }
-  }
-#endif
-
-#ifdef ULTRIX_AUTH
-  {
-    AUTHORIZATION *ap = getauthuid( pass->pw_uid );
-    if (ap)
-      {
-       strcpy( pass->pw_passwd, ap->a_password );
-       endauthent();
-      }
-  }
-#endif
-
-  /* extract relevant info */
-  strcpy(this_user,pass->pw_name);  
-  strcpy(this_salt,pass->pw_passwd);
-#ifdef HPUX
-  /* The crypt on HPUX won't work with more than 2 salt characters. */
-  this_salt[2] = 0;
-#endif /* HPUX */
-  strcpy(this_crypted,pass->pw_passwd);
-  if (!*this_crypted) {
-    if (!lp_null_passwords()) {
-      DEBUG(2,("Disallowing access to %s due to null password\n",this_user));
-      return(False);
-    }
-#ifndef PWDAUTH
-    if (!*password) {
-      DEBUG(3,("Allowing access to %s with null password\n",this_user));
-      return(True);
-    }
-#endif    
-  }
-
-  /* try it as it came to us */
-  if (password_check(password))
-    {
-      update_protected_database(user,True);
-      return(True);
-    }
-
-  /* if the password was given to us with mixed case then we don't
-     need to proceed as we know it hasn't been case modified by the
-     client */
-  if (strhasupper(password) && strhaslower(password))
-    return(False);
-
-  /* make a copy of it */
-  StrnCpy(pass2,password,sizeof(pstring)-1);
-  
-  /* try all lowercase */
-  strlower(password);
-  if (password_check(password))
-    {
-      update_protected_database(user,True);
-      return(True);
-    }
+       /* Ensure the uid's match */
+       if (smb_pass->unix_uid != pass->pw_uid)
+       {
+               DEBUG(3,("Error : UNIX and SMB uids in password files do not match !\n"));
+               return(False);
+       }
 
-  /* give up? */
-  if (level < 1)
-    {
-      update_protected_database(user,False);
+       if (lm_pwd[0] == '\0' && IS_BITS_SET_ALL(smb_pass->acct_ctrl, ACB_PWNOTREQ) && lp_null_passwords())
+       {
+               DEBUG(3,("account for user %s has no password and null passwords are allowed.\n", smb_pass->unix_name));
+               return(True);
+       }
 
-      /* restore it */
-      strcpy(password,pass2);
+       if (smb_password_ok(smb_pass, chal, lm_pwd, nt_pwd))
+       {
+               return(True);
+       }
+       
+       DEBUG(3,("Error smb_password_check failed\n"));
+       return False;
+}
 
-      return(False);
-    }
+/****************************************************************************
+check if a username/password pair is OK either via the system password
+database or the encrypted SMB password database
+return True if the password is correct, False otherwise
+****************************************************************************/
+BOOL password_ok(char *user, char *password, int pwlen, struct passwd *pwd)
+{
+       if (pwlen == 24 || (lp_encrypted_passwords() && (pwlen == 0) && lp_null_passwords()))
+       {
+               /* if 24 bytes long assume it is an encrypted password */
+               uchar challenge[8];
 
-  /* last chance - all combinations of up to level chars upper! */
-  strlower(password);
+               if (!last_challenge(challenge))
+               {
+                       DEBUG(0,("Error: challenge not done for user=%s\n", user));
+                       return False;
+               }
 
-  if (string_combinations(password,password_check,level))
-    {
-      update_protected_database(user,True);
-      return(True);
-    }
+               return pass_check_smb(user, global_myworkgroup,
+                                     challenge, (uchar *)password, (uchar *)password, pwd);
+       } 
 
-  update_protected_database(user,False);
-  
-  /* restore it */
-  strcpy(password,pass2);
-  
-  return(False);
+       return pass_check(user, password, pwlen, pwd, 
+                         lp_update_encrypted() ? 
+                         update_smbpassword_file : NULL);
 }
 
-
-
 /****************************************************************************
 check if a username is valid
 ****************************************************************************/
 BOOL user_ok(char *user,int snum)
 {
-  pstring valid, invalid;
-  BOOL ret;
-
-  StrnCpy(valid, lp_valid_users(snum), sizeof(pstring));
-  StrnCpy(invalid, lp_invalid_users(snum), sizeof(pstring));
-
-  string_sub(valid,"%S",lp_servicename(snum));
-  string_sub(invalid,"%S",lp_servicename(snum));
+       pstring valid, invalid;
+       BOOL ret;
 
-  ret = !user_in_list(user,invalid);
+       StrnCpy(valid, lp_valid_users(snum), sizeof(pstring));
+       StrnCpy(invalid, lp_invalid_users(snum), sizeof(pstring));
 
-  if (ret && valid && *valid)
-    ret = user_in_list(user,valid);
+       string_sub(valid,"%S",lp_servicename(snum));
+       string_sub(invalid,"%S",lp_servicename(snum));
+       
+       ret = !user_in_list(user,invalid);
+       
+       if (ret && valid && *valid) {
+               ret = user_in_list(user,valid);
+       }
 
-  if (ret && lp_onlyuser(snum)) {
-    char *user_list = lp_username(snum);
-    string_sub(user_list,"%S",lp_servicename(snum));
-    ret = user_in_list(user,user_list);
-  }
+       if (ret && lp_onlyuser(snum)) {
+               char *user_list = lp_username(snum);
+               string_sub(user_list,"%S",lp_servicename(snum));
+               ret = user_in_list(user,user_list);
+       }
 
-  return(ret);
+       return(ret);
 }
 
 
@@ -1314,7 +538,7 @@ validate a group username entry. Return the username or NULL
 ****************************************************************************/
 static char *validate_group(char *group,char *password,int pwlen,int snum)
 {
-#ifdef NETGROUP
+#if defined(HAVE_NETGROUP) && defined(HAVE_GETNETGRENT) && defined(HAVE_SETNETGRENT) && defined(HAVE_ENDNETGRENT)
   {
     char *host, *user, *domain;
     setnetgrent(group);
@@ -1331,7 +555,7 @@ static char *validate_group(char *group,char *password,int pwlen,int snum)
   }
 #endif
   
-#if HAVE_GETGRNAM 
+#ifdef HAVE_GETGRNAM 
   {
     struct group *gptr = (struct group *)getgrnam(group);
     char **member;
@@ -1341,7 +565,7 @@ static char *validate_group(char *group,char *password,int pwlen,int snum)
        while (member && *member)
          {
            static fstring name;
-           strcpy(name,*member);
+           fstrcpy(name,*member);
            if (user_ok(name,snum) &&
                password_ok(name,password,pwlen,NULL))
              return(&name[0]);
@@ -1357,7 +581,7 @@ static char *validate_group(char *group,char *password,int pwlen,int snum)
            if (*(pwd->pw_passwd) && pwd->pw_gid == gptr->gr_gid) {
              /* This Entry have PASSWORD and same GID then check pwd */
              if (password_ok(NULL, password, pwlen, pwd)) {
-               strcpy(tm, pwd->pw_name);
+               fstrcpy(tm, pwd->pw_name);
                endpwent ();
                return tm;
              }
@@ -1417,7 +641,7 @@ BOOL authorise_login(int snum,char *user,char *password, int pwlen,
       if (!ok && (vuser != 0) && vuser->guest) {         
        if (user_ok(vuser->name,snum) &&
            password_ok(vuser->name, password, pwlen, NULL)) {
-         strcpy(user, vuser->name);
+         fstrcpy(user, vuser->name);
          vuser->guest = False;
          DEBUG(3,("ACCEPTED: given password with registered user %s\n", user));
          ok = True;
@@ -1426,38 +650,38 @@ BOOL authorise_login(int snum,char *user,char *password, int pwlen,
 
 
       /* now check the list of session users */
-      if (!ok)
-       {
-         char *auser;
-         char *user_list = strdup(session_users);
-         if (!user_list) return(False);
+    if (!ok)
+    {
+      char *auser;
+      char *user_list = strdup(session_users);
+      if (!user_list) return(False);
 
-         for (auser=strtok(user_list,LIST_SEP); 
-              !ok && auser; 
-              auser = strtok(NULL,LIST_SEP))
-           {
-             fstring user2;
-             strcpy(user2,auser);
-             if (!user_ok(user2,snum)) continue;
+      for (auser=strtok(user_list,LIST_SEP); 
+           !ok && auser; 
+           auser = strtok(NULL,LIST_SEP))
+      {
+        fstring user2;
+        fstrcpy(user2,auser);
+        if (!user_ok(user2,snum)) continue;
                  
-             if (password_ok(user2,password, pwlen, NULL)) {
-               ok = True;
-               strcpy(user,user2);
-               DEBUG(3,("ACCEPTED: session list username and given password ok\n"));
-             }
-           }
-         free(user_list);
-       }
-
-      /* check for a previously validated username/password pair */
-      if (!ok && !lp_revalidate(snum) &&
-         (vuser != 0) && !vuser->guest &&
-         user_ok(vuser->name,snum)) {
-       strcpy(user,vuser->name);
-       *guest = False;
-       DEBUG(3,("ACCEPTED: validated uid ok as non-guest\n"));
-       ok = True;
+        if (password_ok(user2,password, pwlen, NULL)) {
+          ok = True;
+          fstrcpy(user,user2);
+          DEBUG(3,("ACCEPTED: session list username and given password ok\n"));
+        }
       }
+      free(user_list);
+    }
+
+    /* check for a previously validated username/password pair */
+    if (!ok && (!lp_revalidate(snum) || lp_security() > SEC_SHARE) &&
+        (vuser != 0) && !vuser->guest &&
+        user_ok(vuser->name,snum)) {
+      fstrcpy(user,vuser->name);
+      *guest = False;
+      DEBUG(3,("ACCEPTED: validated uid ok as non-guest\n"));
+      ok = True;
+    }
 
       /* check for a rhosts entry */
       if (!ok && user_ok(user,snum) && check_hosts_equiv(user)) {
@@ -1483,19 +707,19 @@ BOOL authorise_login(int snum,char *user,char *password, int pwlen,
                if (auser)
                  {
                    ok = True;
-                   strcpy(user,auser);
+                   fstrcpy(user,auser);
                    DEBUG(3,("ACCEPTED: group username and given password ok\n"));
                  }
              }
            else
              {
                fstring user2;
-               strcpy(user2,auser);
+               fstrcpy(user2,auser);
                if (user_ok(user2,snum) && 
                    password_ok(user2,password,pwlen,NULL))
                  {
                    ok = True;
-                   strcpy(user,user2);
+                   fstrcpy(user,user2);
                    DEBUG(3,("ACCEPTED: user list username and given password ok\n"));
                  }
              }
@@ -1510,7 +734,7 @@ BOOL authorise_login(int snum,char *user,char *password, int pwlen,
       StrnCpy(guestname,lp_guestaccount(snum),sizeof(guestname)-1);
       if (Get_Pwnam(guestname,True))
        {
-         strcpy(user,guestname);
+         fstrcpy(user,guestname);
          ok = True;
          DEBUG(3,("ACCEPTED: guest account and guest ok\n"));
        }
@@ -1540,7 +764,7 @@ static BOOL check_user_equiv(char *user, char *remote, char *equiv_file)
   int plus_allowed = 1;
   char *file_host;
   char *file_user;
-  FILE *fp = fopen(equiv_file, "r");
+  FILE *fp = sys_fopen(equiv_file, "r");
   DEBUG(5, ("check_user_equiv %s %s %s\n", user, remote, equiv_file));
   if (! fp) return False;
   while(fgets(buf, sizeof(buf), fp)) 
@@ -1587,7 +811,7 @@ static BOOL check_user_equiv(char *user, char *remote, char *equiv_file)
        {
          BOOL host_ok = False;
 
-#ifdef NETGROUP          
+#if defined(HAVE_NETGROUP) && defined(HAVE_YP_GET_DEFAULT_DOMAIN)
          if (is_group)
            {
              static char *mydomain = NULL;
@@ -1599,7 +823,7 @@ static BOOL check_user_equiv(char *user, char *remote, char *equiv_file)
 #else
          if (is_group)
            {
-             DEBUG(1,("Netgroups not configured - add -DNETGROUP and recompile\n"));
+             DEBUG(1,("Netgroups not configured\n"));
              continue;
            }
 #endif
@@ -1661,7 +885,7 @@ BOOL check_hosts_equiv(char *user)
       char *home = get_home_dir(user);
       if (home) {
              extern int Client;
-             sprintf(rhostsfile, "%s/.rhosts", home);
+             slprintf(rhostsfile, sizeof(rhostsfile)-1, "%s/.rhosts", home);
              if (check_user_equiv(user,client_name(Client),rhostsfile))
                      return(True);
       }
@@ -1671,14 +895,13 @@ BOOL check_hosts_equiv(char *user)
 }
 
 
-static struct cli_state cli;
-
 /****************************************************************************
 return the client state structure
 ****************************************************************************/
 struct cli_state *server_client(void)
 {
-       return &cli;
+       static struct cli_state pw_cli;
+       return &pw_cli;
 }
 
 /****************************************************************************
@@ -1686,65 +909,11 @@ support for server level security
 ****************************************************************************/
 struct cli_state *server_cryptkey(void)
 {
-       fstring desthost;
-       struct in_addr dest_ip;
-       extern fstring local_machine;
-       char *p;
-
-       if (!cli_initialise(&cli))
-               return NULL;
-           
-       for (p=strtok(lp_passwordserver(),LIST_SEP); p ; p = strtok(NULL,LIST_SEP)) {
-               fstrcpy(desthost,p);
-               standard_sub_basic(desthost);
-               strupper(desthost);
-
-                if(!resolve_name( desthost, &dest_ip)) {
-                        DEBUG(1,("server_cryptkey: Can't resolve address for %s\n",p));
-                        continue;
-                }
-
-               if (ismyip(dest_ip)) {
-                       DEBUG(1,("Password server loop - disabling password server %s\n",p));
-                       continue;
-               }
-
-               if (cli_connect(&cli, desthost, &dest_ip)) {
-                       DEBUG(3,("connected to password server %s\n",p));
-                       break;
-               }
-       }
-
-       if (!p) {
-               DEBUG(1,("password server not available\n"));
-               cli_shutdown(&cli);
-               return NULL;
-       }
-
-       if (!cli_session_request(&cli, desthost, 0x20, local_machine)) {
-               DEBUG(1,("%s rejected the session\n",desthost));
-               cli_shutdown(&cli);
-               return NULL;
-       }
-
-       DEBUG(3,("got session\n"));
-
-       if (!cli_negprot(&cli)) {
-               DEBUG(1,("%s rejected the negprot\n",desthost));
-               cli_shutdown(&cli);
-               return NULL;
-       }
-
-       if (cli.protocol < PROTOCOL_LANMAN2 ||
-           !(cli.sec_mode & 1)) {
-               DEBUG(1,("%s isn't in user level security mode\n",desthost));
-               cli_shutdown(&cli);
-               return NULL;
+       if (cli_connect_serverlist(server_client(), lp_passwordserver()))
+       {
+               return server_client();
        }
-
-       DEBUG(3,("password server OK\n"));
-
-       return &cli;
+       return NULL;
 }
 
 /****************************************************************************
@@ -1754,76 +923,246 @@ BOOL server_validate(char *user, char *domain,
                     char *pass, int passlen,
                     char *ntpass, int ntpasslen)
 {
-       extern fstring local_machine;
+  struct cli_state *cli;
+  static unsigned char badpass[24];
+  static BOOL tested_password_server = False;
+  static BOOL bad_password_server = False;
 
-       if (!cli.initialised) {
-               DEBUG(1,("password server %s is not connected\n", cli.desthost));
-               return(False);
-       }  
+  cli = server_client();
 
-       if (!cli_session_setup(&cli, user, pass, passlen, ntpass, ntpasslen, domain)) {
-               DEBUG(1,("password server %s rejected the password\n", cli.desthost));
-               return False;
-       }
+  if (!cli->initialised) {
+    DEBUG(1,("password server %s is not connected\n", cli->desthost));
+    return(False);
+  }  
 
-       /* if logged in as guest then reject */
-       if ((SVAL(cli.inbuf,smb_vwv2) & 1) != 0) {
-               DEBUG(1,("password server %s gave us guest only\n", cli.desthost));
-                cli_ulogoff(&cli);
-               return(False);
-       }
+  if(badpass[0] == 0)
+    memset(badpass, 0x1f, sizeof(badpass));
 
+  if((passlen == sizeof(badpass)) && !memcmp(badpass, pass, passlen)) {
+    /* 
+     * Very unlikely, our random bad password is the same as the users
+     * password. */
+    memset(badpass, badpass[0]+1, sizeof(badpass));
+  }
 
-       if (!cli_send_tconX(&cli, "IPC$", "IPC", "", 1)) {
-               DEBUG(1,("password server %s refused IPC$ connect\n", cli.desthost));
-                cli_ulogoff(&cli);
-               return False;
-       }
+  /*
+   * Attempt a session setup with a totally incorrect password.
+   * If this succeeds with the guest bit *NOT* set then the password
+   * server is broken and is not correctly setting the guest bit. We
+   * need to detect this as some versions of NT4.x are broken. JRA.
+   */
+
+  if(!tested_password_server) {
+    if (cli_session_setup(cli, user, (char *)badpass, sizeof(badpass), 
+                              (char *)badpass, sizeof(badpass), domain)) {
+
+      /*
+       * We connected to the password server so we
+       * can say we've tested it.
+       */
+      tested_password_server = True;
+
+      if ((SVAL(cli->inbuf,smb_vwv2) & 1) == 0) {
+        DEBUG(0,("server_validate: password server %s allows users as non-guest \
+with a bad password.\n", cli->desthost));
+        DEBUG(0,("server_validate: This is broken (and insecure) behaviour. Please do not \
+use this machine as the password server.\n"));
+        cli_ulogoff(cli);
 
         /*
-         * This patch from Rob Nielsen <ran@adc.com> makes doing
-         * the NetWksaUserLogon a dynamic, rather than compile-time
-         * parameter, defaulting to on. This is somewhat dangerous
-         * as it allows people to turn off this neccessary check,
-         * but so many people have had problems with this that I
-         * think it is a neccessary change. JRA.
+         * Password server has the bug.
          */
+        bad_password_server = True;
+        return False;
+      }
+      cli_ulogoff(cli);
+    }
+  } else {
 
-       if (lp_net_wksta_user_logon()) {
-               DEBUG(3,("trying NetWkstaUserLogon with password server %s\n", cli.desthost));
-               if (!cli_NetWkstaUserLogon(&cli,user,local_machine)) {
-                       DEBUG(1,("password server %s failed NetWkstaUserLogon\n", cli.desthost));
-                       cli_tdis(&cli);
-                        cli_ulogoff(&cli);
-                       return False;
-               }
+    /*
+     * We have already tested the password server.
+     * Fail immediately if it has the bug.
+     */
 
-               if (cli.privilages == 0) {
-                       DEBUG(1,("password server %s gave guest privilages\n", cli.desthost));
-                       cli_tdis(&cli);
-                        cli_ulogoff(&cli);
-                       return False;
-               }
+    if(bad_password_server) {
+      DEBUG(0,("server_validate: [1] password server %s allows users as non-guest \
+with a bad password.\n", cli->desthost));
+      DEBUG(0,("server_validate: [1] This is broken (and insecure) behaviour. Please do not \
+use this machine as the password server.\n"));
+      return False;
+    }
+  }
 
-               if (!strequal(cli.eff_name, user)) {
-                       DEBUG(1,("password server %s gave different username %s\n", 
-                               cli.desthost,
-                               cli.eff_name));
-                       cli_tdis(&cli);
-                        cli_ulogoff(&cli);
-                       return False;
-               }
-       }
-        else {
-               DEBUG(3,("skipping NetWkstaUserLogon with password server %s\n", cli.desthost));
-        }
+  /*
+   * Now we know the password server will correctly set the guest bit, or is
+   * not guest enabled, we can try with the real password.
+   */
+
+  if (!cli_session_setup(cli, user, pass, passlen, ntpass, ntpasslen, domain)) {
+    DEBUG(1,("password server %s rejected the password\n", cli->desthost));
+    return False;
+  }
 
-       DEBUG(3,("password server %s accepted the password\n", cli.desthost));
+  /* if logged in as guest then reject */
+  if ((SVAL(cli->inbuf,smb_vwv2) & 1) != 0) {
+    DEBUG(1,("password server %s gave us guest only\n", cli->desthost));
+    cli_ulogoff(cli);
+    return(False);
+  }
 
-       cli_tdis(&cli);
-        cli_ulogoff(&cli);
 
-       return(True);
+  cli_ulogoff(cli);
+
+  return(True);
 }
 
+/***********************************************************************
+ Do the same as security=server, but using NT Domain calls and a session
+ key from the machine password.
+************************************************************************/
+
+BOOL domain_client_validate( char *user, char *domain, 
+                             char *smb_apasswd, int smb_apasslen, 
+                             char *smb_ntpasswd, int smb_ntpasslen)
+{
+  uint16 nt_pipe_fnum;
+  unsigned char local_challenge[8];
+  unsigned char local_lm_response[24];
+  unsigned char local_nt_reponse[24];
+  unsigned char trust_passwd[16];
+  NET_ID_INFO_CTR ctr;
+  NET_USER_INFO_3 info3;
+  struct cli_state cli;
+  uint32 smb_uid_low;
+
+  /* 
+   * Check that the requested domain is not our own machine name.
+   * If it is, we should never check the PDC here, we use our own local
+   * password file.
+   */
+
+  if(strequal( domain, global_myname)) {
+    DEBUG(3,("domain_client_validate: Requested domain was for this machine.\n"));
+    return False;
+  }
+
+  /*
+   * Next, check that the passwords given were encrypted.
+   */
+
+  if(((smb_apasslen != 24) && (smb_apasslen != 0)) || 
+     ((smb_ntpasslen != 24) && (smb_ntpasslen != 0))) {
+
+    /*
+     * Not encrypted - do so.
+     */
+
+    DEBUG(3,("domain_client_validate: User passwords not in encrypted format.\n"));
+    generate_random_buffer( local_challenge, 8, False);
+    SMBencrypt( (uchar *)smb_apasswd, local_challenge, local_lm_response);
+    SMBNTencrypt((uchar *)smb_ntpasswd, local_challenge, local_nt_reponse);
+    smb_apasslen = 24;
+    smb_ntpasslen = 24;
+    smb_apasswd = (char *)local_lm_response;
+    smb_ntpasswd = (char *)local_nt_reponse;
+  } else {
+
+    /*
+     * Encrypted - get the challenge we sent for these
+     * responses.
+     */
+
+    if (!last_challenge(local_challenge)) {
+      DEBUG(0,("domain_client_validate: no challenge done - password failed\n"));
+      return False;
+    }
+  }
+
+  /*
+   * Get the machine account password.
+   */
+  if (!trust_get_passwd( trust_passwd, global_myworkgroup, global_myname))
+  {
+    return False;
+  }
+
+  /*
+   * At this point, smb_apasswd points to the lanman response to
+   * the challenge in local_challenge, and smb_ntpasswd points to
+   * the NT response to the challenge in local_challenge. Ship
+   * these over the secure channel to a domain controller and
+   * see if they were valid.
+   */
+
+       if (!cli_connect_serverlist(&cli, lp_passwordserver()))
+       {
+    DEBUG(0,("domain_client_validate: Domain password server not available.\n"));
+    return False;
+  }
+
+  /*
+   * Ok - we have an anonymous connection to the IPC$ share.
+   * Now start the NT Domain stuff :-).
+   */
+
+  if(cli_nt_session_open(&cli, PIPE_NETLOGON, &nt_pipe_fnum) == False) {
+    DEBUG(0,("domain_client_validate: unable to open the domain client session to \
+machine %s. Error was : %s.\n", cli.desthost, cli_errstr(&cli)));
+    cli_nt_session_close(&cli, nt_pipe_fnum);
+    cli_ulogoff(&cli);
+    cli_shutdown(&cli);
+    return False; 
+  }
+
+  if(cli_nt_setup_creds(&cli, nt_pipe_fnum,
+     cli.mach_acct, trust_passwd, SEC_CHAN_WKSTA) == False)
+  {
+    DEBUG(0,("domain_client_validate: unable to setup the PDC credentials to machine \
+%s. Error was : %s.\n", cli.desthost, cli_errstr(&cli)));
+    cli_nt_session_close(&cli, nt_pipe_fnum);
+    cli_ulogoff(&cli);
+    cli_shutdown(&cli);
+    return False;
+  }
+
+  /* We really don't care what LUID we give the user. */
+  generate_random_buffer( (unsigned char *)&smb_uid_low, 4, False);
+
+  if(cli_nt_login_network(&cli, nt_pipe_fnum, domain, user, smb_uid_low, (char *)local_challenge,
+                          ((smb_apasslen != 0) ? smb_apasswd : NULL),
+                          ((smb_ntpasslen != 0) ? smb_ntpasswd : NULL),
+                          &ctr, &info3) == False) {
+    DEBUG(0,("domain_client_validate: unable to validate password for user %s in domain \
+%s to Domain controller %s. Error was %s.\n", user, domain, cli.desthost, cli_errstr(&cli)));
+    cli_nt_session_close(&cli, nt_pipe_fnum);
+    cli_ulogoff(&cli);
+    cli_shutdown(&cli);
+    return False;
+  }
+
+  /*
+   * Here, if we really want it, we have lots of info about the user in info3.
+   */
 
+#if 0
+  /* 
+   * We don't actually need to do this - plus it fails currently with
+   * NT_STATUS_INVALID_INFO_CLASS - we need to know *exactly* what to
+   * send here. JRA.
+   */
+
+  if(cli_nt_logoff(&cli, nt_pipe_fnum, &ctr) == False) {
+    DEBUG(0,("domain_client_validate: unable to log off user %s in domain \
+%s to Domain controller %s. Error was %s.\n", user, domain, cli.desthost, cli_errstr(&cli)));        
+    cli_nt_session_close(&cli, nt_pipe_fnum);
+    cli_ulogoff(&cli);
+    cli_shutdown(&cli);
+    return False;
+  }
+#endif /* 0 */
+
+  cli_nt_session_close(&cli, nt_pipe_fnum);
+  cli_ulogoff(&cli);
+  cli_shutdown(&cli);
+  return True;
+}