We should use set_domain_offline(domain) function instead of just set staus to offlin...

domain will never goes online again.
From Bo Yang <boyang@novell.com>
Jeremy.

diff --git a/source/nsswitch/winbindd_pam.c b/source/nsswitch/winbindd_pam.c
index e5e55e60f98ad4368076c2690d765fcb74ba3c5d..42540a6d11d2c45b5884f8be20ad0508234bf72c 100644 (file)
--- 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