winbindd: fix trust_is_oubound()
[samba.git] / source3 / winbindd / winbindd_misc.c
index 42ecea2aad705efc76186eca443b90915564c841..d2741cb6b6fc64084db8d066ea5d75086c57f01d 100644 (file)
@@ -22,6 +22,7 @@
 
 #include "includes.h"
 #include "winbindd.h"
+#include "libcli/security/dom_sid.h"
 
 #undef DBGC_CLASS
 #define DBGC_CLASS DBGC_WINBIND
@@ -42,9 +43,9 @@ const char *trust_type_strings[] = {"External",
 
 static enum trust_type get_trust_type(struct winbindd_tdc_domain *domain)
 {
-       if (domain->trust_attribs == NETR_TRUST_ATTRIBUTE_QUARANTINED_DOMAIN)   
+       if (domain->trust_attribs == LSA_TRUST_ATTRIBUTE_QUARANTINED_DOMAIN)
                return EXTERNAL;
-       else if (domain->trust_attribs == NETR_TRUST_ATTRIBUTE_FOREST_TRANSITIVE)
+       else if (domain->trust_attribs == LSA_TRUST_ATTRIBUTE_FOREST_TRANSITIVE)
                return FOREST;
        else if (((domain->trust_flags & NETR_TRUST_FLAG_IN_FOREST) == NETR_TRUST_FLAG_IN_FOREST) &&
            ((domain->trust_flags & NETR_TRUST_FLAG_PRIMARY) == 0x0))
@@ -59,35 +60,54 @@ static const char *get_trust_type_string(struct winbindd_tdc_domain *domain)
 
 static bool trust_is_inbound(struct winbindd_tdc_domain *domain)
 {
-       return (domain->trust_flags == 0x0) ||
-           ((domain->trust_flags & NETR_TRUST_FLAG_IN_FOREST) ==
-            NETR_TRUST_FLAG_IN_FOREST) ||                      
-           ((domain->trust_flags & NETR_TRUST_FLAG_INBOUND) ==
-           NETR_TRUST_FLAG_INBOUND);           
+       if (domain->trust_flags & NETR_TRUST_FLAG_INBOUND) {
+               return true;
+       }
+       return false;
 }
 
 static bool trust_is_outbound(struct winbindd_tdc_domain *domain)
 {
-       return (domain->trust_flags == 0x0) ||
-           ((domain->trust_flags & NETR_TRUST_FLAG_IN_FOREST) ==
-            NETR_TRUST_FLAG_IN_FOREST) ||                      
-           ((domain->trust_flags & NETR_TRUST_FLAG_OUTBOUND) ==
-           NETR_TRUST_FLAG_OUTBOUND);          
+       if (domain->trust_flags & NETR_TRUST_FLAG_OUTBOUND) {
+               return true;
+       }
+       return false;
 }
 
 static bool trust_is_transitive(struct winbindd_tdc_domain *domain)
 {
-       if ((domain->trust_attribs == NETR_TRUST_ATTRIBUTE_NON_TRANSITIVE) ||         
-           (domain->trust_attribs == NETR_TRUST_ATTRIBUTE_QUARANTINED_DOMAIN) ||
-           (domain->trust_attribs == NETR_TRUST_ATTRIBUTE_TREAT_AS_EXTERNAL))
-               return False;
-       return True;
+       bool transitive = false;
+
+       /*
+        * Beware: order matters
+        */
+
+       if (domain->trust_attribs & LSA_TRUST_ATTRIBUTE_WITHIN_FOREST) {
+               transitive = true;
+       }
+
+       if (domain->trust_attribs & LSA_TRUST_ATTRIBUTE_FOREST_TRANSITIVE) {
+               transitive = true;
+       }
+
+       if (domain->trust_attribs & LSA_TRUST_ATTRIBUTE_NON_TRANSITIVE) {
+               transitive = false;
+       }
+
+       if (domain->trust_attribs & LSA_TRUST_ATTRIBUTE_QUARANTINED_DOMAIN) {
+               transitive = false;
+       }
+
+       if (domain->trust_flags & NETR_TRUST_FLAG_PRIMARY) {
+               transitive = true;
+       }
+
+       return transitive;
 }
 
 void winbindd_list_trusted_domains(struct winbindd_cli_state *state)
 {
        struct winbindd_tdc_domain *dom_list = NULL;
-       struct winbindd_tdc_domain *d = NULL;
        size_t num_domains = 0;
        int extra_data_len = 0;
        char *extra_data = NULL;
@@ -110,6 +130,7 @@ void winbindd_list_trusted_domains(struct winbindd_cli_state *state)
        for ( i = 0; i < num_domains; i++ ) {
                struct winbindd_domain *domain;
                bool is_online = true;          
+               struct winbindd_tdc_domain *d = NULL;
 
                d = &dom_list[i];
                domain = find_domain_from_name_noinit(d->domain_name);
@@ -120,7 +141,7 @@ void winbindd_list_trusted_domains(struct winbindd_cli_state *state)
                        extra_data,
                        "%s\\%s\\%s\\%s\\%s\\%s\\%s\\%s\n",
                        d->domain_name,
-                       d->dns_name ? d->dns_name : d->domain_name,
+                       d->dns_name ? d->dns_name : "",
                        sid_string_talloc(state->mem_ctx, &d->sid),
                        get_trust_type_string(d),
                        trust_is_transitive(d) ? "Yes" : "No",
@@ -159,8 +180,7 @@ enum winbindd_result winbindd_dual_list_trusted_domains(struct winbindd_domain *
        DEBUG(3, ("[%5lu]: list trusted domains\n",
                  (unsigned long)state->pid));
 
-       result = domain->methods->trusted_domains(domain, state->mem_ctx,
-                                                 &trusts);
+       result = wb_cache_trusted_domains(domain, state->mem_ctx, &trusts);
 
        if (!NT_STATUS_IS_OK(result)) {
                DEBUG(3, ("winbindd_dual_list_trusted_domains: trusted_domains returned %s\n",
@@ -171,12 +191,21 @@ enum winbindd_result winbindd_dual_list_trusted_domains(struct winbindd_domain *
        extra_data = talloc_strdup(state->mem_ctx, "");
 
        for (i=0; i<trusts.count; i++) {
+
+               if (trusts.array[i].sid == NULL) {
+                       continue;
+               }
+               if (dom_sid_equal(trusts.array[i].sid, &global_sid_NULL)) {
+                       continue;
+               }
+
                extra_data = talloc_asprintf_append_buffer(
-                       extra_data, "%s\\%s\\%s\n",
-                       trusts.array[i].netbios_name,
-                       trusts.array[i].dns_name,
-                       sid_string_talloc(state->mem_ctx,
-                                         trusts.array[i].sid));
+                   extra_data, "%s\\%s\\%s\\%u\\%u\\%u\n",
+                   trusts.array[i].netbios_name, trusts.array[i].dns_name,
+                   sid_string_talloc(state->mem_ctx, trusts.array[i].sid),
+                   trusts.array[i].trust_flags,
+                   (uint32_t)trusts.array[i].trust_type,
+                   trusts.array[i].trust_attributes);
        }
 
        /* add our primary domain */
@@ -191,7 +220,9 @@ enum winbindd_result winbindd_dual_list_trusted_domains(struct winbindd_domain *
        if (state->request->data.list_all_domains && !have_own_domain) {
                extra_data = talloc_asprintf_append_buffer(
                        extra_data, "%s\\%s\\%s\n", domain->name,
-                       domain->alt_name ? domain->alt_name : domain->name,
+                       domain->alt_name != NULL ?
+                               domain->alt_name :
+                               domain->name,
                        sid_string_talloc(state->mem_ctx, &domain->sid));
        }
 
@@ -202,7 +233,7 @@ enum winbindd_result winbindd_dual_list_trusted_domains(struct winbindd_domain *
                extra_data[extra_data_len-1] = '\0';
 
                state->response->extra_data.data = extra_data;
-               state->response->length += extra_data_len+1;
+               state->response->length += extra_data_len;
        }
 
        return WINBINDD_OK;
@@ -266,7 +297,7 @@ void winbindd_domain_info(struct winbindd_cli_state *cli)
         * Send a ping down. This implicitly initializes the domain.
         */
 
-       req = wb_domain_request_send(state, winbind_event_context(),
+       req = wb_domain_request_send(state, server_event_context(),
                                     domain, &state->ping_request);
        if (req == NULL) {
                DEBUG(3, ("wb_domain_request_send failed\n"));
@@ -319,17 +350,17 @@ void winbindd_dc_info(struct winbindd_cli_state *cli)
        struct winbindd_domain *domain;
        char *dc_name, *dc_ip;
 
-       cli->request->domain_name[sizeof(cli->request->domain_name-1)] = '\0';
+       cli->request->domain_name[sizeof(cli->request->domain_name)-1] = '\0';
 
        DEBUG(3, ("[%5lu]: domain_info [%s]\n", (unsigned long)cli->pid,
                  cli->request->domain_name));
 
        if (cli->request->domain_name[0] != '\0') {
-               domain = find_domain_from_name_noinit(
+               domain = find_trust_from_name_noinit(
                        cli->request->domain_name);
-               DEBUG(10, ("Could not find domain %s\n",
-                          cli->request->domain_name));
                if (domain == NULL) {
+                       DEBUG(10, ("Could not find domain %s\n",
+                                  cli->request->domain_name));
                        request_error(cli);
                        return;
                }
@@ -380,8 +411,8 @@ void winbindd_info(struct winbindd_cli_state *state)
 
 void winbindd_interface_version(struct winbindd_cli_state *state)
 {
-       DEBUG(3, ("[%5lu]: request interface version\n",
-                 (unsigned long)state->pid));
+       DEBUG(3, ("[%5lu]: request interface version (version = %d)\n",
+                 (unsigned long)state->pid, WINBIND_INTERFACE_VERSION));
 
        state->response->data.interface_version = WINBIND_INTERFACE_VERSION;
        request_ok(state);
@@ -404,7 +435,7 @@ void winbindd_netbios_name(struct winbindd_cli_state *state)
        DEBUG(3, ("[%5lu]: request netbios name\n",
                  (unsigned long)state->pid));
 
-       fstrcpy(state->response->data.netbios_name, global_myname());
+       fstrcpy(state->response->data.netbios_name, lp_netbios_name());
        request_ok(state);
 }