windows: improved logging from NPLogonNotify

Improve the detail and formatting of the windows event log
entries.

Change-Id: I088f6d35afb91e146e8e6030f29dec39301b020b
Reviewed-on: http://gerrit.openafs.org/4556
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
This commit is contained in:
Jeffrey Altman 2011-04-25 21:23:34 -04:00 committed by Derrick Brashear
parent 63fd8a3e5b
commit df85c8095b

View File

@ -644,7 +644,7 @@ GetDomainLogonOptions( PLUID lpLogonId, char * username, char * domain, LogonOpt
} }
if (hkTemp) { if (hkTemp) {
CHAR * thesecells = NULL; CHAR * thesecells = NULL, *p;
/* dwSize still has the size of the required buffer in bytes. */ /* dwSize still has the size of the required buffer in bytes. */
thesecells = malloc(dwSize*2); thesecells = malloc(dwSize*2);
@ -658,9 +658,16 @@ GetDomainLogonOptions( PLUID lpLogonId, char * username, char * domain, LogonOpt
goto doneTheseCells; goto doneTheseCells;
} }
DebugEvent("Found TheseCells [%s]", thesecells); /* TheseCells is a REG_MULTI_SZ */
opt->theseCells = thesecells; if ( thesecells && thesecells[0]) {
thesecells = NULL; for ( p=thesecells; *p; p += (strlen(p) + 1)) {
DebugEvent("Found TheseCells [%s]", p);
}
opt->theseCells = thesecells;
thesecells = NULL;
} else {
DebugEvent("TheseCells [REG_MULTI_SZ] not found");
}
doneTheseCells: doneTheseCells:
if (thesecells) free(thesecells); if (thesecells) free(thesecells);
@ -857,7 +864,7 @@ DWORD APIENTRY NPLogonNotify(
!UnicodeStringToANSI(IL->LogonDomainName, logonDomain, MAX_DOMAIN_LENGTH)) !UnicodeStringToANSI(IL->LogonDomainName, logonDomain, MAX_DOMAIN_LENGTH))
return 0; return 0;
/* Make sure AD-DOMAINS sent from login that is sent to us is striped */ /* Make sure AD-DOMAINS sent from login that is sent to us is stripped */
ctemp = strchr(uname, '@'); ctemp = strchr(uname, '@');
if (ctemp) { if (ctemp) {
*ctemp = 0; *ctemp = 0;
@ -886,7 +893,7 @@ DWORD APIENTRY NPLogonNotify(
if (retryInterval < sleepInterval) if (retryInterval < sleepInterval)
sleepInterval = retryInterval; sleepInterval = retryInterval;
DebugEvent("Got logon script: %S",opt.logonScript); DebugEvent("Got logon script: [%S]", opt.logonScript);
afsWillAutoStart = AFSWillAutoStart(); afsWillAutoStart = AFSWillAutoStart();
@ -905,7 +912,7 @@ DWORD APIENTRY NPLogonNotify(
/* Get cell name if doing integrated logon. /* Get cell name if doing integrated logon.
We might overwrite this if we are logging into an AD realm and we find out that We might overwrite this if we are logging into an AD realm and we find out that
the user's home dir is in some other cell. */ the user's home dir is in some other cell. */
DebugEvent("About to call cm_GetRootCellName(%s)",cell); DebugEvent("About to call cm_GetRootCellName()");
code = cm_GetRootCellName(cell); code = cm_GetRootCellName(cell);
if (code < 0) { if (code < 0) {
DebugEvent0("Unable to obtain Root Cell"); DebugEvent0("Unable to obtain Root Cell");
@ -913,7 +920,7 @@ DWORD APIENTRY NPLogonNotify(
reason = "unknown cell"; reason = "unknown cell";
code=0; code=0;
} else { } else {
DebugEvent("Cell is %s",cell); DebugEvent("Default cell is %s", cell);
} }
/* We get the user's home directory path, if applicable, though we can't lookup the /* We get the user's home directory path, if applicable, though we can't lookup the