From: Jeffrey Altman <jaltman@secure-endpoints.com>
Date: Thu, 12 Jan 2006 05:54:23 +0000 (+0000)
Subject: windows-cmfinduserbyname-20060112
X-Git-Tag: openafs-devel-1_5_0~112
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=af202cac551ad541a0a1ca4525b03c2ec05f6507;p=packages%2Fo%2Fopenafs.git

windows-cmfinduserbyname-20060112

FIXES 25347

Fix invalid access after freeing newly allocated data structure
---

diff --git a/src/WINNT/afsd/smb3.c b/src/WINNT/afsd/smb3.c
index fefc6e57a..afbcfc6c8 100644
--- a/src/WINNT/afsd/smb3.c
+++ b/src/WINNT/afsd/smb3.c
@@ -7149,6 +7149,7 @@ void smb3_Init()
 cm_user_t *smb_FindCMUserByName(char *usern, char *machine)
 {
     smb_username_t *unp;
+    cm_user_t *     userp;
 
     unp = smb_FindUserByName(usern, machine, SMB_FLAG_CREATE);
     if (!unp->userp) {
@@ -7159,8 +7160,9 @@ cm_user_t *smb_FindCMUserByName(char *usern, char *machine)
     }  else	{
         osi_Log2(smb_logp,"smb_FindCMUserByName Not found name[%s] machine[%s]",osi_LogSaveString(smb_logp,usern),osi_LogSaveString(smb_logp,machine));
     }
-    cm_HoldUser(unp->userp);
+    userp = unp->userp;
+    cm_HoldUser(userp);
     smb_ReleaseUsername(unp);
-    return unp->userp;
+    return userp;
 }