winbindd: the ad trusted_domains call should return talloced strings.

Guenther

diff --git a/source/winbindd/winbindd_ads.c b/source/winbindd/winbindd_ads.c
index 53ea3e148c16baff9c5477496bd36da092445277..94e3bad7b2331fcc761226dbcc3f030a4fd86237 100644 (file)
--- a/source/winbindd/winbindd_ads.c
+++ b/source/winbindd/winbindd_ads.c
@@ -1239,8 +1239,11 @@ static NTSTATUS trusted_domains(struct winbindd_domain *domain,
                                continue;                               
                        }
                        
-                       (*names)[ret_count] = CONST_DISCARD(char *, trusts.array[i].netbios_name);
-                       (*alt_names)[ret_count] = CONST_DISCARD(char *, trusts.array[i].dns_name);
+                       (*names)[ret_count] = talloc_strdup(mem_ctx, trusts.array[i].netbios_name);
+                       (*alt_names)[ret_count] = talloc_strdup(mem_ctx, trusts.array[i].dns_name);
+                       if ((*names)[ret_count] == NULL) {
+                               return NT_STATUS_NO_MEMORY;
+                       }
                        if (trusts.array[i].sid) {
                                sid_copy(&(*dom_sids)[ret_count], trusts.array[i].sid);
                        } else {