s3: Simplify pdb_set_plaintext_passwd: pwhistory==NULL can not happen anymore
authorVolker Lendecke <vl@samba.org>
Mon, 14 Dec 2009 18:18:09 +0000 (19:18 +0100)
committerMichael Adam <obnox@samba.org>
Thu, 7 Jan 2010 10:07:54 +0000 (11:07 +0100)
source3/passdb/pdb_get_set.c

index eed3591fd68c6ea3f6044682bb8a92dcba1c9b61..005cf46a6251fab05e8b30677723737836bf74a5 100644 (file)
@@ -1061,32 +1061,27 @@ bool pdb_set_plaintext_passwd(struct samu *sampass, const char *plaintext)
                pwhistory = new_history;
        }
 
-       if (pwhistory != NULL) {
-               /*
-                * Make room for the new password in the history list.
-                */
-               if (pwHistLen > 1) {
-                       memmove(&pwhistory[PW_HISTORY_ENTRY_LEN], pwhistory,
-                               (pwHistLen-1)*PW_HISTORY_ENTRY_LEN );
-               }
-               /*
-                * Create the new salt as the first part of the
-                * history entry.
-                */
-               generate_random_buffer(pwhistory, PW_HISTORY_SALT_LEN);
+       /*
+        * Make room for the new password in the history list.
+        */
+       if (pwHistLen > 1) {
+               memmove(&pwhistory[PW_HISTORY_ENTRY_LEN], pwhistory,
+                       (pwHistLen-1)*PW_HISTORY_ENTRY_LEN );
+       }
 
-               /*
-                * Generate the md5 hash of the salt+new password as
-                * the second part of the history entry.
-                */
+       /*
+        * Create the new salt as the first part of the history entry.
+        */
+       generate_random_buffer(pwhistory, PW_HISTORY_SALT_LEN);
+
+       /*
+        * Generate the md5 hash of the salt+new password as the
+        * second part of the history entry.
+        */
+       E_md5hash(pwhistory, new_nt_p16, &pwhistory[PW_HISTORY_SALT_LEN]);
+
+       pdb_set_pw_history(sampass, pwhistory, pwHistLen, PDB_CHANGED);
 
-               E_md5hash(pwhistory, new_nt_p16,
-                         &pwhistory[PW_HISTORY_SALT_LEN]);
-               pdb_set_pw_history(sampass, pwhistory, pwHistLen, PDB_CHANGED);
-       } else {
-               DEBUG (10,("pdb_get_set.c: pdb_set_plaintext_passwd: "
-                          "pwhistory was NULL!\n"));
-       }
        return True;
 }