s3-winbind: Fix null pointer dereference in store_memory_creds().

Found by Coverity.

Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Günther Deschner <gd@samba.org>

diff --git a/source3/winbindd/winbindd_cred_cache.c b/source3/winbindd/winbindd_cred_cache.c
index e8bbf9c82413923442a02b3bc5bf3ce57c47be3a..01ca3ff1afc0e245f975ea15249e0057054b84a3 100644 (file)
--- a/source3/winbindd/winbindd_cred_cache.c
+++ b/source3/winbindd/winbindd_cred_cache.c
@@ -846,11 +846,11 @@ static NTSTATUS store_memory_creds(struct WINBINDD_MEMORY_CREDS *memcredp,
        DEBUG(10,("mlocked memory: %p\n", memcredp->nt_hash));
 #endif
 
-       /* Create and store the password hashes. */
-       E_md4hash(pass, memcredp->nt_hash);
-       E_deshash(pass, memcredp->lm_hash);
-
        if (pass) {
+               /* Create and store the password hashes. */
+               E_md4hash(pass, memcredp->nt_hash);
+               E_deshash(pass, memcredp->lm_hash);
+
                memcredp->pass = (char *)memcredp->lm_hash + LM_HASH_LEN;
                memcpy(memcredp->pass, pass,
                       memcredp->len - NT_HASH_LEN - LM_HASH_LEN);