From: Jeffrey Altman Date: Tue, 19 Jun 2012 01:39:51 +0000 (-0400) Subject: Windows: afslogon MapAuthError X-Git-Tag: upstream/1.8.0_pre1^2~2287 X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=7adb93cf51d56936d51b70758a959342728fea56;p=packages%2Fo%2Fopenafs.git Windows: afslogon MapAuthError return WN_NET_ERROR (ERROR_UNEXP_NET_ERR) to indicate a problem with this network provider when an error occurs instead of returning WN_NET_SUCCESS. Change-Id: Ideaab8b5043e9033adc19efa158c95415f561d0e Reviewed-on: http://gerrit.openafs.org/7629 Tested-by: BuildBot Reviewed-by: Jeffrey Altman Tested-by: Jeffrey Altman --- diff --git a/src/WINNT/afsd/afslogon.c b/src/WINNT/afsd/afslogon.c index 68d6ba633..294b7c2ee 100644 --- a/src/WINNT/afsd/afslogon.c +++ b/src/WINNT/afsd/afslogon.c @@ -226,14 +226,17 @@ DWORD MapAuthError(DWORD code) switch (code) { /* Unfortunately, returning WN_NO_NETWORK results in the MPR abandoning * logon scripts for all credential managers, although they will still - * receive logon notifications. Since we don't want this, we return - * WN_SUCCESS. This is highly undesirable, but we also don't want to - * break other network providers. + * receive logon notifications. + * + * Instead return WN_NET_ERROR (ERROR_UNEXP_NET_ERR) to indicate a + * problem with this network. */ - /* case KTC_NOCM: + case KTC_NOCM: case KTC_NOCMRPC: - return WN_NO_NETWORK; */ - default: return WN_SUCCESS; + return WN_NET_ERROR; + + default: + return WN_SUCCESS; } }