Tiny logic simplification: Remove an unnecessary else branch

(This used to be commit 6009ae329375b1c40e3d00df977ddccc8b5cc176)

diff --git a/source3/winbindd/idmap_ad.c b/source3/winbindd/idmap_ad.c
index 2e79413ffc2f8b2835faa23981ff669b32100983..b99da57a4a72e6e3b58258a85ca4b6a55c1cfe9c 100644 (file)
--- a/source3/winbindd/idmap_ad.c
+++ b/source3/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) {