mirror of
https://github.com/samba-team/samba.git
synced 2025-03-20 22:50:26 +03:00
rpcclient: fix variable initialisation and add parenthesis to if clauses
Just a few README.Coding fixes. Bug: https://bugzilla.samba.org/show_bug.cgi?id=13286 Signed-off-by: Ralph Boehme <slow@samba.org> Reviewed-by: Stefan Metzmacher <metze@samba.org> (cherry picked from commit 9ccc6eef145c1f67e24cbb1c21402714f612c607)
This commit is contained in:
parent
92f0b55258
commit
0d4e2c8829
@ -272,8 +272,8 @@ static NTSTATUS cmd_lsa_lookup_names_level(struct rpc_pipe_client *cli,
|
||||
{
|
||||
struct policy_handle pol;
|
||||
NTSTATUS status, result;
|
||||
struct dom_sid *sids;
|
||||
enum lsa_SidType *types;
|
||||
struct dom_sid *sids = NULL;
|
||||
enum lsa_SidType *types = NULL;
|
||||
int i, level;
|
||||
struct dcerpc_binding_handle *b = cli->binding_handle;
|
||||
|
||||
@ -285,9 +285,9 @@ static NTSTATUS cmd_lsa_lookup_names_level(struct rpc_pipe_client *cli,
|
||||
status = rpccli_lsa_open_policy(cli, mem_ctx, True,
|
||||
SEC_FLAG_MAXIMUM_ALLOWED,
|
||||
&pol);
|
||||
|
||||
if (!NT_STATUS_IS_OK(status))
|
||||
if (!NT_STATUS_IS_OK(status)) {
|
||||
goto done;
|
||||
}
|
||||
|
||||
level = atoi(argv[1]);
|
||||
|
||||
@ -296,7 +296,9 @@ static NTSTATUS cmd_lsa_lookup_names_level(struct rpc_pipe_client *cli,
|
||||
|
||||
if (!NT_STATUS_IS_OK(status) && NT_STATUS_V(status) !=
|
||||
NT_STATUS_V(STATUS_SOME_UNMAPPED))
|
||||
{
|
||||
goto done;
|
||||
}
|
||||
|
||||
status = NT_STATUS_OK;
|
||||
|
||||
|
Loading…
x
Reference in New Issue
Block a user