[SCM] Samba Shared Repository - branch v3-0-test updated - release-3-0-29-38-g1ad5cd9

[Jeremy Allison]

The branch, v3-0-test has been updated
       via  1ad5cd93e69d8b82a71690d16df7e5cb2078bb65 (commit)
      from  3fa483a54d68fcebb204f532eaff01e1dff19f76 (commit)

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=v3-0-test


- Log -----------------------------------------------------------------
commit 1ad5cd93e69d8b82a71690d16df7e5cb2078bb65
Author: Jeremy Allison <jra at samba.org>
Date:   Fri Jun 13 09:43:34 2008 -0700

    We should use set_domain_offline(domain) function instead of just set staus to offline. Otherwise,
    domain will never goes online again.
    From Bo Yang <boyang at novell.com>
    Jeremy.

-----------------------------------------------------------------------

Summary of changes:
 source/nsswitch/winbindd_pam.c |    6 +++++-
 1 files changed, 5 insertions(+), 1 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source/nsswitch/winbindd_pam.c b/source/nsswitch/winbindd_pam.c
index e5e55e6..42540a6 100644
--- a/source/nsswitch/winbindd_pam.c
+++ b/source/nsswitch/winbindd_pam.c
@@ -1379,7 +1379,11 @@ enum winbindd_result winbindd_dual_pam_auth(struct winbindd_domain *domain,
 		    NT_STATUS_EQUAL(result, NT_STATUS_IO_TIMEOUT) ||
 		    NT_STATUS_EQUAL(result, NT_STATUS_DOMAIN_CONTROLLER_NOT_FOUND)) {
 			DEBUG(10,("winbindd_dual_pam_auth_kerberos setting domain to offline\n"));
-			domain->online = False;
+			/* Use set_domain_offline() instead of
+			 * just set status offline, otherwise,
+			 * domain will never goes online again
+			 * --- BoYang */
+			set_domain_offline(domain);
 		}
 
 		/* there are quite some NT_STATUS errors where there is no


-- 
Samba Shared Repository