libnet_join: streamline logic of libnet_join_post_processing()
authorMichael Adam <obnox@samba.org>
Fri, 29 Aug 2008 15:55:28 +0000 (17:55 +0200)
committerMichael Adam <obnox@samba.org>
Fri, 29 Aug 2008 16:00:51 +0000 (18:00 +0200)
Michael
(This used to be commit 81cc1af1e699e454fbb1d12636d002f845231006)

source3/libnet/libnet_join.c

index b34b4872f492c59a86e93629eeee2f390b165100..a39dee676fc7b0cb1706f4d409d21bcc3f71bb18 100644 (file)
@@ -1503,20 +1503,22 @@ static WERROR libnet_join_post_processing(TALLOC_CTX *mem_ctx,
                return werr;
        }
 
-       if (r->in.join_flags & WKSSVC_JOIN_FLAGS_JOIN_TYPE) {
-               saf_store(r->in.domain_name, r->in.dc_name);
+       if (!(r->in.join_flags & WKSSVC_JOIN_FLAGS_JOIN_TYPE)) {
+               return WERR_OK;
+       }
+
+       saf_store(r->in.domain_name, r->in.dc_name);
 
 #ifdef WITH_ADS
-               if (r->out.domain_is_ad) {
-                       ADS_STATUS ads_status;
+       if (r->out.domain_is_ad) {
+               ADS_STATUS ads_status;
 
-                       ads_status  = libnet_join_post_processing_ads(mem_ctx, r);
-                       if (!ADS_ERR_OK(ads_status)) {
-                               return WERR_GENERAL_FAILURE;
-                       }
+               ads_status  = libnet_join_post_processing_ads(mem_ctx, r);
+               if (!ADS_ERR_OK(ads_status)) {
+                       return WERR_GENERAL_FAILURE;
                }
-#endif /* WITH_ADS */
        }
+#endif /* WITH_ADS */
 
        libnet_join_add_dom_rids_to_builtins(r->out.domain_sid);