Avoid overriding default ccache for ads operations.
[kai/samba.git] / source3 / winbindd / winbindd_ads.c
index 039c59e3398d46b0d9a8030ff6e919fbdf0dabb9..628fd1c3f2790c7902b0e8651240c6779a1ce921 100644 (file)
 
 #include "includes.h"
 #include "winbindd.h"
-#include "../librpc/gen_ndr/cli_netlogon.h"
+#include "rpc_client/rpc_client.h"
+#include "../librpc/gen_ndr/ndr_netlogon_c.h"
+#include "../libds/common/flags.h"
+#include "ads.h"
+#include "secrets.h"
+#include "../libcli/ldap/ldap_ndr.h"
+#include "../libcli/security/security.h"
+#include "../libds/common/flag_mapping.h"
+#include "passdb.h"
 
 #ifdef HAVE_ADS
 
@@ -70,15 +78,15 @@ static ADS_STRUCT *ads_cached_connection(struct winbindd_domain *domain)
                }
        }
 
-       /* we don't want this to affect the users ccache */
-       setenv("KRB5CCNAME", "MEMORY:winbind_ccache", 1);
-
        ads = ads_init(domain->alt_name, domain->name, NULL);
        if (!ads) {
                DEBUG(1,("ads_init for domain %s failed\n", domain->name));
                return NULL;
        }
 
+       /* we don't want ads operations to affect the default ccache */
+       ads->auth.ccache_name = SMB_STRDUP("MEMORY:winbind_ccache");
+
        /* the machine acct password might have change - fetch it every time */
 
        SAFE_FREE(ads->auth.password);
@@ -91,7 +99,10 @@ static ADS_STRUCT *ads_cached_connection(struct winbindd_domain *domain)
                        return NULL;
                }
                ads->auth.realm = SMB_STRDUP( ads->server.realm );
-               strupper_m( ads->auth.realm );
+               if (!strupper_m( ads->auth.realm )) {
+                       ads_destroy( &ads );
+                       return NULL;
+               }
        }
        else {
                struct winbindd_domain *our_domain = domain;
@@ -106,7 +117,10 @@ static ADS_STRUCT *ads_cached_connection(struct winbindd_domain *domain)
 
                if ( our_domain->alt_name[0] != '\0' ) {
                        ads->auth.realm = SMB_STRDUP( our_domain->alt_name );
-                       strupper_m( ads->auth.realm );
+                       if (!strupper_m( ads->auth.realm )) {
+                               ads_destroy( &ads );
+                               return NULL;
+                       }
                }
                else
                        ads->auth.realm = SMB_STRDUP( lp_realm() );
@@ -152,7 +166,7 @@ static ADS_STRUCT *ads_cached_connection(struct winbindd_domain *domain)
 static NTSTATUS query_user_list(struct winbindd_domain *domain,
                               TALLOC_CTX *mem_ctx,
                               uint32 *num_entries, 
-                              struct wbint_userinfo **info)
+                              struct wbint_userinfo **pinfo)
 {
        ADS_STRUCT *ads = NULL;
        const char *attrs[] = { "*", NULL };
@@ -180,8 +194,12 @@ static NTSTATUS query_user_list(struct winbindd_domain *domain,
        }
 
        rc = ads_search_retry(ads, &res, "(objectCategory=user)", attrs);
-       if (!ADS_ERR_OK(rc) || !res) {
+       if (!ADS_ERR_OK(rc)) {
                DEBUG(1,("query_user_list ads_search: %s\n", ads_errstr(rc)));
+               status = ads_ntstatus(rc);
+       } else if (!res) {
+               DEBUG(1,("query_user_list ads_search returned NULL res\n"));
+
                goto done;
        }
 
@@ -191,23 +209,18 @@ static NTSTATUS query_user_list(struct winbindd_domain *domain,
                goto done;
        }
 
-       (*info) = TALLOC_ZERO_ARRAY(mem_ctx, struct wbint_userinfo, count);
-       if (!*info) {
+       (*pinfo) = talloc_zero_array(mem_ctx, struct wbint_userinfo, count);
+       if (!*pinfo) {
                status = NT_STATUS_NO_MEMORY;
                goto done;
        }
 
-       i = 0;
+       count = 0;
 
        for (msg = ads_first_entry(ads, res); msg; msg = ads_next_entry(ads, msg)) {
-               const char *name;
-               const char *gecos = NULL;
-               const char *homedir = NULL;
-               const char *shell = NULL;
+               struct wbint_userinfo *info = &((*pinfo)[count]);
                uint32 group;
                uint32 atype;
-               DOM_SID user_sid;
-               gid_t primary_gid = (gid_t)-1;
 
                if (!ads_pull_uint32(ads, msg, "sAMAccountType", &atype) ||
                    ds_atype_map(atype) != SID_NAME_USER) {
@@ -215,46 +228,58 @@ static NTSTATUS query_user_list(struct winbindd_domain *domain,
                        continue;
                }
 
-               name = ads_pull_username(ads, mem_ctx, msg);
-
-               if ( ads_pull_sid( ads, msg, "objectSid", &user_sid ) ) {
-                       status = nss_get_info_cached( domain, &user_sid, mem_ctx, 
-                                              ads, msg, &homedir, &shell, &gecos,
-                                              &primary_gid );
-               }
-
-               if (gecos == NULL) {
-                       gecos = ads_pull_string(ads, mem_ctx, msg, "name");
-               }
+               info->acct_name = ads_pull_username(ads, mem_ctx, msg);
+               info->full_name = ads_pull_string(ads, mem_ctx, msg, "name");
+               info->homedir = NULL;
+               info->shell = NULL;
+               info->primary_gid = (gid_t)-1;
 
                if (!ads_pull_sid(ads, msg, "objectSid",
-                                 &(*info)[i].user_sid)) {
-                       DEBUG(1,("No sid for %s !?\n", name));
+                                 &info->user_sid)) {
+                       DEBUG(1, ("No sid for %s !?\n", info->acct_name));
                        continue;
                }
+
                if (!ads_pull_uint32(ads, msg, "primaryGroupID", &group)) {
-                       DEBUG(1,("No primary group for %s !?\n", name));
+                       DEBUG(1, ("No primary group for %s !?\n",
+                                 info->acct_name));
                        continue;
                }
+               sid_compose(&info->group_sid, &domain->sid, group);
 
-               (*info)[i].acct_name = name;
-               (*info)[i].full_name = gecos;
-               (*info)[i].homedir = homedir;
-               (*info)[i].shell = shell;
-               (*info)[i].primary_gid = primary_gid;
-               sid_compose(&(*info)[i].group_sid, &domain->sid, group);
-               i++;
+               count += 1;
+       }
+
+       (*num_entries) = count;
+       ads_msgfree(ads, res);
+
+       for (i=0; i<count; i++) {
+               struct wbint_userinfo *info = &((*pinfo)[i]);
+               const char *gecos = NULL;
+               gid_t primary_gid = (gid_t)-1;
+
+               status = nss_get_info_cached(domain, &info->user_sid, mem_ctx,
+                                            &info->homedir, &info->shell,
+                                            &gecos, &primary_gid);
+               if (!NT_STATUS_IS_OK(status)) {
+                       /*
+                        * Deliberately ignore this error, there might be more
+                        * users to fill
+                        */
+                       continue;
+               }
+
+               if (gecos != NULL) {
+                       info->full_name = gecos;
+               }
+               info->primary_gid = primary_gid;
        }
 
-       (*num_entries) = i;
        status = NT_STATUS_OK;
 
        DEBUG(3,("ads query_user_list gave %d entries\n", (*num_entries)));
 
 done:
-       if (res) 
-               ads_msgfree(ads, res);
-
        return status;
 }
 
@@ -262,7 +287,7 @@ done:
 static NTSTATUS enum_dom_groups(struct winbindd_domain *domain,
                                TALLOC_CTX *mem_ctx,
                                uint32 *num_entries, 
-                               struct acct_info **info)
+                               struct wb_acct_info **info)
 {
        ADS_STRUCT *ads = NULL;
        const char *attrs[] = {"userPrincipalName", "sAMAccountName",
@@ -325,9 +350,13 @@ static NTSTATUS enum_dom_groups(struct winbindd_domain *domain,
        }
 
        rc = ads_search_retry(ads, &res, filter, attrs);
-       if (!ADS_ERR_OK(rc) || !res) {
+       if (!ADS_ERR_OK(rc)) {
+               status = ads_ntstatus(rc);
                DEBUG(1,("enum_dom_groups ads_search: %s\n", ads_errstr(rc)));
                goto done;
+       } else if (!res) {
+               DEBUG(1,("enum_dom_groups ads_search returned NULL res\n"));
+               goto done;
        }
 
        count = ads_count_replies(ads, res);
@@ -336,7 +365,7 @@ static NTSTATUS enum_dom_groups(struct winbindd_domain *domain,
                goto done;
        }
 
-       (*info) = TALLOC_ZERO_ARRAY(mem_ctx, struct acct_info, count);
+       (*info) = talloc_zero_array(mem_ctx, struct wb_acct_info, count);
        if (!*info) {
                status = NT_STATUS_NO_MEMORY;
                goto done;
@@ -346,7 +375,7 @@ static NTSTATUS enum_dom_groups(struct winbindd_domain *domain,
 
        for (msg = ads_first_entry(ads, res); msg; msg = ads_next_entry(ads, msg)) {
                char *name, *gecos;
-               DOM_SID sid;
+               struct dom_sid sid;
                uint32 rid;
 
                name = ads_pull_username(ads, mem_ctx, msg);
@@ -384,7 +413,7 @@ done:
 static NTSTATUS enum_local_groups(struct winbindd_domain *domain,
                                TALLOC_CTX *mem_ctx,
                                uint32 *num_entries, 
-                               struct acct_info **info)
+                               struct wb_acct_info **info)
 {
        /*
         * This is a stub function only as we returned the domain 
@@ -407,7 +436,7 @@ static NTSTATUS name_to_sid(struct winbindd_domain *domain,
                            const char *domain_name,
                            const char *name,
                            uint32_t flags,
-                           DOM_SID *sid,
+                           struct dom_sid *sid,
                            enum lsa_SidType *type)
 {
        return reconnect_methods.name_to_sid(domain, mem_ctx,
@@ -418,7 +447,7 @@ static NTSTATUS name_to_sid(struct winbindd_domain *domain,
 /* convert a domain SID to a user or group name - use rpc methods */
 static NTSTATUS sid_to_name(struct winbindd_domain *domain,
                            TALLOC_CTX *mem_ctx,
-                           const DOM_SID *sid,
+                           const struct dom_sid *sid,
                            char **domain_name,
                            char **name,
                            enum lsa_SidType *type)
@@ -430,7 +459,7 @@ static NTSTATUS sid_to_name(struct winbindd_domain *domain,
 /* convert a list of rids to names - use rpc methods */
 static NTSTATUS rids_to_names(struct winbindd_domain *domain,
                              TALLOC_CTX *mem_ctx,
-                             const DOM_SID *sid,
+                             const struct dom_sid *sid,
                              uint32 *rids,
                              size_t num_rids,
                              char **domain_name,
@@ -450,7 +479,7 @@ static NTSTATUS rids_to_names(struct winbindd_domain *domain,
 /* Lookup user information from a rid */
 static NTSTATUS query_user(struct winbindd_domain *domain, 
                           TALLOC_CTX *mem_ctx, 
-                          const DOM_SID *sid, 
+                          const struct dom_sid *sid,
                           struct wbint_userinfo *info)
 {
        ADS_STRUCT *ads = NULL;
@@ -463,17 +492,18 @@ static NTSTATUS query_user(struct winbindd_domain *domain,
        uint32 group_rid;
        NTSTATUS status = NT_STATUS_UNSUCCESSFUL;
        struct netr_SamInfo3 *user = NULL;
-       gid_t gid;
+       gid_t gid = -1;
+       int ret;
+       char *ads_name;
 
        DEBUG(3,("ads: query_user\n"));
 
        info->homedir = NULL;
        info->shell = NULL;
-       info->primary_gid = (gid_t)-1;
 
        /* try netsamlogon cache first */
 
-       if ( (user = netsamlogon_cache_get( mem_ctx, sid )) != NULL ) 
+       if (winbindd_use_cache() && (user = netsamlogon_cache_get( mem_ctx, sid )) != NULL )
        {
                DEBUG(5,("query_user: Cache lookup succeeded for %s\n", 
                         sid_string_dbg(sid)));
@@ -484,7 +514,7 @@ static NTSTATUS query_user(struct winbindd_domain *domain,
                info->acct_name = talloc_strdup(mem_ctx, user->base.account_name.string);
                info->full_name = talloc_strdup(mem_ctx, user->base.full_name.string);
 
-               nss_get_info_cached( domain, sid, mem_ctx, NULL, NULL, 
+               nss_get_info_cached( domain, sid, mem_ctx,
                              &info->homedir, &info->shell, &info->full_name, 
                              &gid );
                info->primary_gid = gid;
@@ -506,75 +536,93 @@ static NTSTATUS query_user(struct winbindd_domain *domain,
 
                /* Assume "Domain Users" for the primary group */
 
-               sid_compose(&info->group_sid, &domain->sid, DOMAIN_GROUP_RID_USERS );
+               sid_compose(&info->group_sid, &domain->sid, DOMAIN_RID_USERS );
 
                /* Try to fill in what the nss_info backend can do */
 
-               nss_get_info_cached( domain, sid, mem_ctx, NULL, NULL, 
+               nss_get_info_cached( domain, sid, mem_ctx,
                              &info->homedir, &info->shell, &info->full_name, 
                              &gid);
                info->primary_gid = gid;
 
-               status = NT_STATUS_OK;
-               goto done;
+               return NT_STATUS_OK;
        }
 
        /* no cache...do the query */
 
        if ( (ads = ads_cached_connection(domain)) == NULL ) {
                domain->last_status = NT_STATUS_SERVER_DISABLED;
-               goto done;
+               return NT_STATUS_SERVER_DISABLED;
        }
 
-       sidstr = sid_binstring(talloc_tos(), sid);
-       if (asprintf(&ldap_exp, "(objectSid=%s)", sidstr) == -1) {
-               status = NT_STATUS_NO_MEMORY;
-               goto done;
+       sidstr = ldap_encode_ndr_dom_sid(talloc_tos(), sid);
+
+       ret = asprintf(&ldap_exp, "(objectSid=%s)", sidstr);
+       TALLOC_FREE(sidstr);
+       if (ret == -1) {
+               return NT_STATUS_NO_MEMORY;
        }
        rc = ads_search_retry(ads, &msg, ldap_exp, attrs);
-       free(ldap_exp);
-       TALLOC_FREE(sidstr);
-       if (!ADS_ERR_OK(rc) || !msg) {
+       SAFE_FREE(ldap_exp);
+       if (!ADS_ERR_OK(rc)) {
                DEBUG(1,("query_user(sid=%s) ads_search: %s\n",
                         sid_string_dbg(sid), ads_errstr(rc)));
-               goto done;
+               return ads_ntstatus(rc);
+       } else if (!msg) {
+               DEBUG(1,("query_user(sid=%s) ads_search returned NULL res\n",
+                        sid_string_dbg(sid)));
+               return NT_STATUS_INTERNAL_ERROR;
        }
 
        count = ads_count_replies(ads, msg);
        if (count != 1) {
                DEBUG(1,("query_user(sid=%s): Not found\n",
                         sid_string_dbg(sid)));
-               goto done;
+               ads_msgfree(ads, msg);
+               return NT_STATUS_NO_SUCH_USER;
        }
 
        info->acct_name = ads_pull_username(ads, mem_ctx, msg);
 
-       nss_get_info_cached( domain, sid, mem_ctx, ads, msg, 
-                     &info->homedir, &info->shell, &info->full_name, 
-                     &gid);
-       info->primary_gid = gid;
-
-       if (info->full_name == NULL) {
-               info->full_name = ads_pull_string(ads, mem_ctx, msg, "name");
-       }
-
        if (!ads_pull_uint32(ads, msg, "primaryGroupID", &group_rid)) {
                DEBUG(1,("No primary group for %s !?\n",
                         sid_string_dbg(sid)));
-               goto done;
+               ads_msgfree(ads, msg);
+               return NT_STATUS_NO_SUCH_USER;
        }
-
        sid_copy(&info->user_sid, sid);
        sid_compose(&info->group_sid, &domain->sid, group_rid);
 
+       /*
+        * We have to fetch the "name" attribute before doing the
+        * nss_get_info_cached call. nss_get_info_cached might destroy
+        * the ads struct, potentially invalidating the ldap message.
+        */
+       ads_name = ads_pull_string(ads, mem_ctx, msg, "name");
+
+       ads_msgfree(ads, msg);
+       msg = NULL;
+
+       status = nss_get_info_cached( domain, sid, mem_ctx,
+                     &info->homedir, &info->shell, &info->full_name, 
+                     &gid);
+       info->primary_gid = gid;
+       if (!NT_STATUS_IS_OK(status)) {
+               DEBUG(1, ("nss_get_info_cached failed: %s\n",
+                         nt_errstr(status)));
+               return status;
+       }
+
+       if (info->full_name == NULL) {
+               info->full_name = ads_name;
+       } else {
+               TALLOC_FREE(ads_name);
+       }
+
        status = NT_STATUS_OK;
 
        DEBUG(3,("ads query_user gave %s\n", info->acct_name));
-done:
-       if (msg) 
-               ads_msgfree(ads, msg);
-
-       return status;
+       return NT_STATUS_OK;
 }
 
 /* Lookup groups a user is a member of - alternate method, for when
@@ -582,8 +630,8 @@ done:
 static NTSTATUS lookup_usergroups_member(struct winbindd_domain *domain,
                                         TALLOC_CTX *mem_ctx,
                                         const char *user_dn, 
-                                        DOM_SID *primary_group,
-                                        size_t *p_num_groups, DOM_SID **user_sids)
+                                        struct dom_sid *primary_group,
+                                        uint32_t *p_num_groups, struct dom_sid **user_sids)
 {
        ADS_STATUS rc;
        NTSTATUS status = NT_STATUS_UNSUCCESSFUL;
@@ -594,7 +642,7 @@ static NTSTATUS lookup_usergroups_member(struct winbindd_domain *domain,
        ADS_STRUCT *ads;
        const char *group_attrs[] = {"objectSid", NULL};
        char *escaped_dn;
-       size_t num_groups = 0;
+       uint32_t num_groups = 0;
 
        DEBUG(3,("ads: lookup_usergroups_member\n"));
 
@@ -632,11 +680,15 @@ static NTSTATUS lookup_usergroups_member(struct winbindd_domain *domain,
 
        rc = ads_search_retry(ads, &res, ldap_exp, group_attrs);
 
-       if (!ADS_ERR_OK(rc) || !res) {
+       if (!ADS_ERR_OK(rc)) {
                DEBUG(1,("lookup_usergroups ads_search member=%s: %s\n", user_dn, ads_errstr(rc)));
                return ads_ntstatus(rc);
+       } else if (!res) {
+               DEBUG(1,("lookup_usergroups ads_search returned NULL res\n"));
+               return NT_STATUS_INTERNAL_ERROR;
        }
 
+
        count = ads_count_replies(ads, res);
 
        *user_sids = NULL;
@@ -652,7 +704,7 @@ static NTSTATUS lookup_usergroups_member(struct winbindd_domain *domain,
        if (count > 0) {
                for (msg = ads_first_entry(ads, res); msg;
                     msg = ads_next_entry(ads, msg)) {
-                       DOM_SID group_sid;
+                       struct dom_sid group_sid;
 
                        if (!ads_pull_sid(ads, msg, "objectSid", &group_sid)) {
                                DEBUG(1,("No sid for this group ?!?\n"));
@@ -689,16 +741,16 @@ done:
 static NTSTATUS lookup_usergroups_memberof(struct winbindd_domain *domain,
                                           TALLOC_CTX *mem_ctx,
                                           const char *user_dn,
-                                          DOM_SID *primary_group,
-                                          size_t *p_num_groups,
-                                          DOM_SID **user_sids)
+                                          struct dom_sid *primary_group,
+                                          uint32_t *p_num_groups,
+                                          struct dom_sid **user_sids)
 {
        ADS_STATUS rc;
        NTSTATUS status = NT_STATUS_UNSUCCESSFUL;
        ADS_STRUCT *ads;
        const char *attrs[] = {"memberOf", NULL};
-       size_t num_groups = 0;
-       DOM_SID *group_sids = NULL;
+       uint32_t num_groups = 0;
+       struct dom_sid *group_sids = NULL;
        int i;
        char **strings = NULL;
        size_t num_strings = 0, num_sids = 0;
@@ -739,7 +791,7 @@ static NTSTATUS lookup_usergroups_memberof(struct winbindd_domain *domain,
                goto done;
        }
 
-       group_sids = TALLOC_ZERO_ARRAY(mem_ctx, DOM_SID, num_strings + 1);
+       group_sids = talloc_zero_array(mem_ctx, struct dom_sid, num_strings + 1);
        if (!group_sids) {
                status = NT_STATUS_NO_MEMORY;
                goto done;
@@ -801,8 +853,8 @@ done:
 /* Lookup groups a user is a member of. */
 static NTSTATUS lookup_usergroups(struct winbindd_domain *domain,
                                  TALLOC_CTX *mem_ctx,
-                                 const DOM_SID *sid, 
-                                 uint32 *p_num_groups, DOM_SID **user_sids)
+                                 const struct dom_sid *sid,
+                                 uint32 *p_num_groups, struct dom_sid **user_sids)
 {
        ADS_STRUCT *ads = NULL;
        const char *attrs[] = {"tokenGroups", "primaryGroupID", NULL};
@@ -810,12 +862,12 @@ static NTSTATUS lookup_usergroups(struct winbindd_domain *domain,
        int count;
        LDAPMessage *msg = NULL;
        char *user_dn = NULL;
-       DOM_SID *sids;
+       struct dom_sid *sids;
        int i;
-       DOM_SID primary_group;
+       struct dom_sid primary_group;
        uint32 primary_group_rid;
        NTSTATUS status = NT_STATUS_UNSUCCESSFUL;
-       size_t num_groups = 0;
+       uint32_t num_groups = 0;
 
        DEBUG(3,("ads: lookup_usergroups\n"));
        *p_num_groups = 0;
@@ -880,8 +932,7 @@ static NTSTATUS lookup_usergroups(struct winbindd_domain *domain,
                goto done;
        }
 
-       sid_copy(&primary_group, &domain->sid);
-       sid_append_rid(&primary_group, primary_group_rid);
+       sid_compose(&primary_group, &domain->sid, primary_group_rid);
 
        count = ads_pull_sids(ads, mem_ctx, msg, "tokenGroups", &sids);
 
@@ -901,7 +952,7 @@ static NTSTATUS lookup_usergroups(struct winbindd_domain *domain,
                status = lookup_usergroups_memberof(domain, mem_ctx, user_dn,
                                                    &primary_group,
                                                    &num_groups, user_sids);
-               *p_num_groups = (uint32)num_groups;
+               *p_num_groups = num_groups;
                if (NT_STATUS_IS_OK(status)) {
                        goto done;
                }
@@ -912,7 +963,7 @@ static NTSTATUS lookup_usergroups(struct winbindd_domain *domain,
                status = lookup_usergroups_member(domain, mem_ctx, user_dn, 
                                                  &primary_group,
                                                  &num_groups, user_sids);
-               *p_num_groups = (uint32)num_groups;
+               *p_num_groups = num_groups;
                goto done;
        }
 
@@ -953,7 +1004,7 @@ done:
 /* Lookup aliases a user is member of - use rpc methods */
 static NTSTATUS lookup_useraliases(struct winbindd_domain *domain,
                                   TALLOC_CTX *mem_ctx,
-                                  uint32 num_sids, const DOM_SID *sids,
+                                  uint32 num_sids, const struct dom_sid *sids,
                                   uint32 *num_aliases, uint32 **alias_rids)
 {
        return reconnect_methods.lookup_useraliases(domain, mem_ctx,
@@ -967,10 +1018,10 @@ static NTSTATUS lookup_useraliases(struct winbindd_domain *domain,
  */
 static NTSTATUS lookup_groupmem(struct winbindd_domain *domain,
                                TALLOC_CTX *mem_ctx,
-                               const DOM_SID *group_sid,
+                               const struct dom_sid *group_sid,
                                enum lsa_SidType type,
                                uint32 *num_names,
-                               DOM_SID **sid_mem, char ***names,
+                               struct dom_sid **sid_mem, char ***names,
                                uint32 **name_types)
 {
        ADS_STATUS rc;
@@ -982,7 +1033,7 @@ static NTSTATUS lookup_groupmem(struct winbindd_domain *domain,
        int i;
        size_t num_members = 0;
        ads_control args;
-       DOM_SID *sid_mem_nocache = NULL;
+       struct dom_sid *sid_mem_nocache = NULL;
        char **names_nocache = NULL;
        enum lsa_SidType *name_types_nocache = NULL;
        char **domains_nocache = NULL;     /* only needed for rpccli_lsa_lookup_sids */
@@ -1014,7 +1065,7 @@ static NTSTATUS lookup_groupmem(struct winbindd_domain *domain,
                goto done;
        }
 
-       if ((sidbinstr = sid_binstring(talloc_tos(), group_sid)) == NULL) {
+       if ((sidbinstr = ldap_encode_ndr_dom_sid(talloc_tos(), group_sid)) == NULL) {
                status = NT_STATUS_NO_MEMORY;
                goto done;
        }
@@ -1056,10 +1107,10 @@ static NTSTATUS lookup_groupmem(struct winbindd_domain *domain,
         * cache. Only the rest is passed to the lsa_lookup_sids call. */
 
        if (num_members) {
-               (*sid_mem) = TALLOC_ZERO_ARRAY(mem_ctx, DOM_SID, num_members);
-               (*names) = TALLOC_ZERO_ARRAY(mem_ctx, char *, num_members);
-               (*name_types) = TALLOC_ZERO_ARRAY(mem_ctx, uint32, num_members);
-               (sid_mem_nocache) = TALLOC_ZERO_ARRAY(tmp_ctx, DOM_SID, num_members);
+               (*sid_mem) = talloc_zero_array(mem_ctx, struct dom_sid, num_members);
+               (*names) = talloc_zero_array(mem_ctx, char *, num_members);
+               (*name_types) = talloc_zero_array(mem_ctx, uint32, num_members);
+               (sid_mem_nocache) = talloc_zero_array(tmp_ctx, struct dom_sid, num_members);
 
                if ((members == NULL) || (*sid_mem == NULL) ||
                    (*names == NULL) || (*name_types == NULL) ||
@@ -1079,7 +1130,7 @@ static NTSTATUS lookup_groupmem(struct winbindd_domain *domain,
        for (i=0; i<num_members; i++) {
                enum lsa_SidType name_type;
                char *name, *domain_name;
-               DOM_SID sid;
+               struct dom_sid sid;
 
                rc = ads_get_sid_from_extended_dn(tmp_ctx, members[i], args.val,
                    &sid);
@@ -1257,33 +1308,25 @@ static NTSTATUS password_policy(struct winbindd_domain *domain,
 /* get a list of trusted domains */
 static NTSTATUS trusted_domains(struct winbindd_domain *domain,
                                TALLOC_CTX *mem_ctx,
-                               uint32 *num_domains,
-                               char ***names,
-                               char ***alt_names,
-                               DOM_SID **dom_sids)
+                               struct netr_DomainTrustList *trusts)
 {
        NTSTATUS                result = NT_STATUS_UNSUCCESSFUL;
-       struct netr_DomainTrustList trusts;
+       WERROR werr;
        int                     i;
        uint32                  flags;  
        struct rpc_pipe_client *cli;
-       uint32                 fr_flags = (NETR_TRUST_FLAG_IN_FOREST | NETR_TRUST_FLAG_TREEROOT);
        int ret_count;
+       struct dcerpc_binding_handle *b;
 
        DEBUG(3,("ads: trusted_domains\n"));
 
-       *num_domains = 0;
-       *alt_names   = NULL;
-       *names       = NULL;
-       *dom_sids    = NULL;
+       ZERO_STRUCTP(trusts);
 
        /* If this is our primary domain or a root in our forest,
           query for all trusts.  If not, then just look for domain
           trusts in the target forest */
 
-       if ( domain->primary ||
-               ((domain->domain_flags&fr_flags) == fr_flags) ) 
-       {
+       if (domain->primary || domain_is_forest_root(domain)) {
                flags = NETR_TRUST_FLAG_OUTBOUND |
                        NETR_TRUST_FLAG_INBOUND |
                        NETR_TRUST_FLAG_IN_FOREST;
@@ -1300,39 +1343,29 @@ static NTSTATUS trusted_domains(struct winbindd_domain *domain,
                return NT_STATUS_UNSUCCESSFUL;
        }
 
-       result = rpccli_netr_DsrEnumerateDomainTrusts(cli, mem_ctx,
+       b = cli->binding_handle;
+
+       result = dcerpc_netr_DsrEnumerateDomainTrusts(b, mem_ctx,
                                                      cli->desthost,
                                                      flags,
-                                                     &trusts,
-                                                     NULL);
+                                                     trusts,
+                                                     &werr);
        if (!NT_STATUS_IS_OK(result)) {
                return result;
        }
-       if (trusts.count == 0) {
-               return NT_STATUS_OK;
-       }
-
-       /* Allocate memory for trusted domain names and sids */
-
-       if ( !(*names = TALLOC_ARRAY(mem_ctx, char *, trusts.count)) ) {
-               DEBUG(0, ("trusted_domains: out of memory\n"));
-               return NT_STATUS_NO_MEMORY;
-       }
 
-       if ( !(*alt_names = TALLOC_ARRAY(mem_ctx, char *, trusts.count)) ) {
-               DEBUG(0, ("trusted_domains: out of memory\n"));
-               return NT_STATUS_NO_MEMORY;
+       if (!W_ERROR_IS_OK(werr)) {
+               return werror_to_ntstatus(werr);
        }
-
-       if ( !(*dom_sids = TALLOC_ARRAY(mem_ctx, DOM_SID, trusts.count)) ) {
-               DEBUG(0, ("trusted_domains: out of memory\n"));
-               return NT_STATUS_NO_MEMORY;
+       if (trusts->count == 0) {
+               return NT_STATUS_OK;
        }
 
        /* Copy across names and sids */
 
        ret_count = 0;
-       for (i = 0; i < trusts.count; i++) {
+       for (i = 0; i < trusts->count; i++) {
+               struct netr_DomainTrust *trust = &trusts->array[i];
                struct winbindd_domain d;
 
                ZERO_STRUCT(d);
@@ -1344,39 +1377,23 @@ static NTSTATUS trusted_domains(struct winbindd_domain *domain,
                 * by the DC.
                 */
 
-               if ((trusts.array[i].trust_attributes
+               if ((trust->trust_attributes
                     == NETR_TRUST_ATTRIBUTE_QUARANTINED_DOMAIN) &&
                    !domain->primary )
                {
                        DEBUG(10,("trusted_domains: Skipping external trusted "
                                  "domain %s because it is outside of our "
                                  "primary domain\n",
-                                 trusts.array[i].netbios_name));
+                                 trust->netbios_name));
                        continue;
                }
 
-               /*
-                * We must check that the SID of each trusted domain
-                * was returned to work around a bug in Windows:
-                * http://support.microsoft.com/kb/922832
-                */
-
-               (*names)[ret_count] = CONST_DISCARD(
-                       char *, trusts.array[i].netbios_name);
-               (*alt_names)[ret_count] = CONST_DISCARD(
-                       char *, trusts.array[i].dns_name);
-               if (trusts.array[i].sid) {
-                       sid_copy(&(*dom_sids)[ret_count], trusts.array[i].sid);
-               } else {
-                       sid_copy(&(*dom_sids)[ret_count], &global_sid_NULL);
-               }
-
                /* add to the trusted domain cache */
 
-               fstrcpy( d.name,  trusts.array[i].netbios_name);
-               fstrcpy( d.alt_name, trusts.array[i].dns_name);
-               if (trusts.array[i].sid) {
-                       sid_copy( &d.sid, trusts.array[i].sid);
+               fstrcpy(d.name, trust->netbios_name);
+               fstrcpy(d.alt_name, trust->dns_name);
+               if (trust->sid) {
+                       sid_copy(&d.sid, trust->sid);
                } else {
                        sid_copy(&d.sid, &global_sid_NULL);
                }
@@ -1387,14 +1404,13 @@ static NTSTATUS trusted_domains(struct winbindd_domain *domain,
                                  "trust flags for domain %s\n",
                                  domain->name, d.alt_name));
 
-                       d.domain_flags = trusts.array[i].trust_flags;
-                       d.domain_type = trusts.array[i].trust_type;
-                       d.domain_trust_attribs =
-                               trusts.array[i].trust_attributes;
+                       d.domain_flags = trust->trust_flags;
+                       d.domain_type = trust->trust_type;
+                       d.domain_trust_attribs = trust->trust_attributes;
 
                        wcache_tdc_add_domain( &d );
                        ret_count++;
-               } else if ( (domain->domain_flags&fr_flags) == fr_flags ) {
+               } else if (domain_is_forest_root(domain)) {
                        /* Check if we already have this record. If
                         * we are following our forest root that is not
                         * our primary domain, we want to keep trust
@@ -1403,16 +1419,16 @@ static NTSTATUS trusted_domains(struct winbindd_domain *domain,
                        struct winbindd_tdc_domain *exist = NULL;
 
                        exist = wcache_tdc_fetch_domain(
-                               NULL, trusts.array[i].netbios_name);
+                               talloc_tos(), trust->netbios_name);
                        if (!exist) {
                                DEBUG(10,("trusted_domains(ads):  Searching "
                                          "trusted domain list of %s and "
                                          "storing trust flags for domain "
                                          "%s\n", domain->name, d.alt_name));
-                               d.domain_flags = trusts.array[i].trust_flags;
-                               d.domain_type = trusts.array[i].trust_type;
+                               d.domain_flags = trust->trust_flags;
+                               d.domain_type = trust->trust_type;
                                d.domain_trust_attribs =
-                                       trusts.array[i].trust_attributes;
+                                       trust->trust_attributes;
 
                                wcache_tdc_add_domain( &d );
                                ret_count++;
@@ -1433,7 +1449,8 @@ static NTSTATUS trusted_domains(struct winbindd_domain *domain,
                                  "trust flags for domain %s\n",
                                  domain->name, d.alt_name));
 
-                       parent = wcache_tdc_fetch_domain(NULL, domain->name);
+                       parent = wcache_tdc_fetch_domain(talloc_tos(),
+                                                        domain->name);
                        if (parent) {
                                d.domain_flags = parent->trust_flags;
                                d.domain_type  = parent->trust_type;
@@ -1450,8 +1467,6 @@ static NTSTATUS trusted_domains(struct winbindd_domain *domain,
                        ret_count++;
                }
        }
-       *num_domains = ret_count;
-
        return result;
 }