diff options
| author | Volker Lendecke <vlendec@samba.org> | 2007-07-29 09:23:55 +0000 |
|---|---|---|
| committer | Gerald (Jerry) Carter <jerry@samba.org> | 2007-10-10 12:29:01 -0500 |
| commit | b876bca6bcdb091c4f4f11af3bd758c8ce4db50a (patch) | |
| tree | d7ca1f5ba04bcfcf4cca546f9b9777c4e6ad4ffe /source/rpcclient | |
| parent | 411a1a2037886f934c6522ddef7a04d5c4a1fa69 (diff) | |
| download | samba-b876bca6bcdb091c4f4f11af3bd758c8ce4db50a.tar.gz samba-b876bca6bcdb091c4f4f11af3bd758c8ce4db50a.tar.xz samba-b876bca6bcdb091c4f4f11af3bd758c8ce4db50a.zip | |
r24068: Fix Coverity id 366 (dead code). We have checked for argc >= 3 above.
Diffstat (limited to 'source/rpcclient')
| -rw-r--r-- | source/rpcclient/cmd_samr.c | 15 |
1 files changed, 6 insertions, 9 deletions
diff --git a/source/rpcclient/cmd_samr.c b/source/rpcclient/cmd_samr.c index 2a35b0cf192..b5ccab88863 100644 --- a/source/rpcclient/cmd_samr.c +++ b/source/rpcclient/cmd_samr.c @@ -1873,15 +1873,12 @@ static NTSTATUS cmd_samr_lookup_rids(struct rpc_pipe_client *cli, /* Look up rids */ num_rids = argc - 2; - if (num_rids) { - if ((rids = TALLOC_ARRAY(mem_ctx, uint32, num_rids)) == NULL) { - rpccli_samr_close(cli, mem_ctx, &domain_pol); - rpccli_samr_close(cli, mem_ctx, &connect_pol); - result = NT_STATUS_NO_MEMORY; - goto done; - } - } else { - rids = NULL; + + if ((rids = TALLOC_ARRAY(mem_ctx, uint32, num_rids)) == NULL) { + rpccli_samr_close(cli, mem_ctx, &domain_pol); + rpccli_samr_close(cli, mem_ctx, &connect_pol); + result = NT_STATUS_NO_MEMORY; + goto done; } for (i = 0; i < argc - 2; i++) |