Make a talloc'ed copy of this strings so we can pass the right kind of pointer

to the function.  This fixes a nice little segfault the brute-force-casting
created. :-)

Andrew Bartlett

diff --git a/source/rpcclient/cmd_lsarpc.c b/source/rpcclient/cmd_lsarpc.c
index f6d0569de93d191074870d3b9db34875793bebd4..67efbb1ff739faf8a5d7aace6589301a9194b289 100644 (file)
--- a/source/rpcclient/cmd_lsarpc.c
+++ b/source/rpcclient/cmd_lsarpc.c
@@ -81,6 +81,7 @@ static NTSTATUS cmd_lsa_lookup_names(struct cli_state *cli,
        uint32 *types;
        int num_names, i;
        fstring name, domain;
+       const char *name2, *domain2;
 
        if (argc == 1) {
                printf("Usage: %s [name1 [name2 [...]]]\n", argv[0]);
@@ -98,8 +99,11 @@ static NTSTATUS cmd_lsa_lookup_names(struct cli_state *cli,
 
        split_domain_name(argv[1], domain, name);
 
+       name2 = talloc_strdup(mem_ctx, name);
+       domain2 = talloc_strdup(mem_ctx, domain);
+
        result = cli_lsa_lookup_names(cli, mem_ctx, &pol, argc - 1, 
-                                     (const char**)&domain, (const char**)&name, &sids, 
+                                     &domain2, &name2, &sids, 
                                      &types, &num_names);
 
        if (!NT_STATUS_IS_OK(result))