From: Stefan Metzmacher <metze@samba.org>
Date: Mon, 20 Mar 2017 11:56:00 +0000 (+0100)
Subject: s4:rpc_server/lsa: remove unused 'status' variable in dcesrv_lsa_LookupSids_common()
X-Git-Tag: tevent-0.9.36~37
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=e8a0223633fd2e6ebb3d864570b76932bc3e293a;p=thirdparty%2Fsamba.git

s4:rpc_server/lsa: remove unused 'status' variable in dcesrv_lsa_LookupSids_common()

Bug: https://bugzilla.samba.org/show_bug.cgi?id=13286

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
---

diff --git a/source4/rpc_server/lsa/lsa_lookup.c b/source4/rpc_server/lsa/lsa_lookup.c
index 219e30194b7..a8a136b1266 100644
--- a/source4/rpc_server/lsa/lsa_lookup.c
+++ b/source4/rpc_server/lsa/lsa_lookup.c
@@ -593,7 +593,6 @@ static NTSTATUS dcesrv_lsa_LookupSids_common(struct dcesrv_call_state *dce_call,
 					     struct lsa_LookupSids2 *r)
 {
 	struct lsa_RefDomainList *domains = NULL;
-	NTSTATUS status = NT_STATUS_OK;
 	uint32_t i;
 
 	*r->out.domains = NULL;
@@ -641,14 +640,12 @@ static NTSTATUS dcesrv_lsa_LookupSids_common(struct dcesrv_call_state *dce_call,
 
 		if (sid_str == NULL) {
 			r->out.names->names[i].name.string = "(SIDERROR)";
-			status = STATUS_SOME_UNMAPPED;
 			continue;
 		}
 
 		status2 = dcesrv_lsa_lookup_sid(state, mem_ctx, sid, sid_str, 
 						&authority_name, &name, &rtype);
 		if (!NT_STATUS_IS_OK(status2)) {
-			status = STATUS_SOME_UNMAPPED;
 			continue;
 		}
 
@@ -675,7 +672,7 @@ static NTSTATUS dcesrv_lsa_LookupSids_common(struct dcesrv_call_state *dce_call,
 		return STATUS_SOME_UNMAPPED;
 	}
 
-	return status;
+	return NT_STATUS_OK;
 }
 
 /*