Revert "Tiny logic simplification: Remove an unnecessary else branch"

This reverts commit 6009ae329375b1c40e3d00df977ddccc8b5cc176.

diff --git a/source/winbindd/idmap_ad.c b/source/winbindd/idmap_ad.c
index b99da57a4a72e6e3b58258a85ca4b6a55c1cfe9c..2e79413ffc2f8b2835faa23981ff669b32100983 100644 (file)
--- a/source/winbindd/idmap_ad.c
+++ b/source/winbindd/idmap_ad.c
@@ -77,15 +77,15 @@ static ADS_STRUCT *ad_idmap_cached_connection_internal(void)
 
                if ( ads->config.realm && (expire > time(NULL))) {
                        return ads;
+               } else {
+                       /* we own this ADS_STRUCT so make sure it goes away */
+                       DEBUG(7,("Deleting expired krb5 credential cache\n"));
+                       ads->is_mine = True;
+                       ads_destroy( &ads );
+                       ads_kdestroy(WINBIND_CCACHE_NAME);
+                       ad_idmap_ads = NULL;
+                       TALLOC_FREE( ad_schema );                       
                }
-
-               /* we own this ADS_STRUCT so make sure it goes away */
-               DEBUG(7,("Deleting expired krb5 credential cache\n"));
-               ads->is_mine = True;
-               ads_destroy( &ads );
-               ads_kdestroy(WINBIND_CCACHE_NAME);
-               ad_idmap_ads = NULL;
-               TALLOC_FREE( ad_schema );
        }
 
        if (!local) {