s3 build: Fix "assignment discards qualifiers from pointer target type" warnings
authorTim Prouty <tprouty@samba.org>
Mon, 2 Feb 2009 07:59:53 +0000 (23:59 -0800)
committerTim Prouty <tprouty@samba.org>
Mon, 2 Feb 2009 08:03:09 +0000 (00:03 -0800)
source3/winbindd/winbindd_group.c
source3/winbindd/winbindd_user.c

index 48e65779022a5c18d20004a277cd119a1fdb961d..9d9b264124f7fed1d93bf1f710da33754dee36e6 100644 (file)
@@ -1306,7 +1306,6 @@ void winbindd_getgrent(struct winbindd_cli_state *state)
                char *gr_mem;
                DOM_SID group_sid;
                struct winbindd_domain *domain;
-               char *domain_name_idmap;
 
                /* Do we need to fetch another chunk of groups? */
 
@@ -1354,11 +1353,8 @@ void winbindd_getgrent(struct winbindd_cli_state *state)
                sid_copy(&group_sid, &domain->sid);
                sid_append_rid(&group_sid, name_list[ent->sam_entry_index].rid);
 
-               domain_name_idmap = domain->have_idmap_config
-                                 ? domain->name
-                                 : "";
-
-               if (!NT_STATUS_IS_OK(idmap_sid_to_gid(domain_name_idmap,
+               if (!NT_STATUS_IS_OK(idmap_sid_to_gid(domain->have_idmap_config
+                                                     ? domain->name : "",
                                                      &group_sid, &group_gid)))
                {
                        union unid_t id;
index b01e1847f26369d5a3b14735a8bed2b4a34882ee..62fd4d7f0758f31b9f51b5492dee3142f48bbc0c 100644 (file)
@@ -76,7 +76,6 @@ static bool winbindd_fill_pwent(TALLOC_CTX *ctx, char *dom_name, char *user_name
        char *mapped_name = NULL;
        struct winbindd_domain *domain = NULL;
        NTSTATUS nt_status = NT_STATUS_UNSUCCESSFUL;
-       char *dom_name_idmap = "";
 
        if (!pw || !dom_name || !user_name)
                return False;
@@ -88,13 +87,10 @@ static bool winbindd_fill_pwent(TALLOC_CTX *ctx, char *dom_name, char *user_name
                nt_status = NT_STATUS_NO_SUCH_DOMAIN;
        }
 
-       if (domain->have_idmap_config) {
-               dom_name_idmap = dom_name;
-       }
-
        /* Resolve the uid number */
 
-       if (!NT_STATUS_IS_OK(idmap_sid_to_uid(dom_name_idmap, user_sid,
+       if (!NT_STATUS_IS_OK(idmap_sid_to_uid(domain->have_idmap_config ?
+                                             dom_name : "", user_sid,
                                              &pw->pw_uid))) {
                DEBUG(1, ("error getting user id for sid %s\n",
                          sid_string_dbg(user_sid)));
@@ -103,7 +99,8 @@ static bool winbindd_fill_pwent(TALLOC_CTX *ctx, char *dom_name, char *user_name
 
        /* Resolve the gid number */
 
-       if (!NT_STATUS_IS_OK(idmap_sid_to_gid(dom_name_idmap, group_sid,
+       if (!NT_STATUS_IS_OK(idmap_sid_to_gid(domain->have_idmap_config ?
+                                             dom_name : "", group_sid,
                                              &pw->pw_gid))) {
                DEBUG(1, ("error getting group id for sid %s\n",
                          sid_string_dbg(group_sid)));