From: Jeffrey Altman <jaltman@secure-endpoints.com>
Date: Tue, 1 Sep 2009 00:06:27 +0000 (-0400)
Subject: Windows: integrated logon
X-Git-Tag: openafs-devel-1_5_63~62
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=95df1a47d1e40eb9d007853c99826432e7ac4016;p=packages%2Fo%2Fopenafs.git

Windows: integrated logon

The previous fix prevented a null pointer dereference but
forgot to set 'effDomain' to the domain specified by the
caller if it exists and is not the local machine name.

LICENSE MIT

FIXES 125325

Reviewed-on: http://gerrit.openafs.org/390
Tested-by: Asanka Herath <asanka@secure-endpoints.com>
Reviewed-by: Asanka Herath <asanka@secure-endpoints.com>
Tested-by: Jeffrey Altman <jaltman@openafs.org>
Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
---

diff --git a/src/WINNT/afsd/afslogon.c b/src/WINNT/afsd/afslogon.c
index 43363a36a..0554eed6d 100644
--- a/src/WINNT/afsd/afslogon.c
+++ b/src/WINNT/afsd/afslogon.c
@@ -375,6 +375,8 @@ GetDomainLogonOptions( PLUID lpLogonId, char * username, char * domain, LogonOpt
                 opt->flags = LOGON_FLAG_LOCAL;
             }
         }
+        if (effDomain == NULL)
+            effDomain = domain;
     }
 
     rv = RegOpenKeyEx( HKEY_LOCAL_MACHINE, AFSREG_CLT_SVC_PARAM_SUBKEY, 0, KEY_READ, &hkParm );