r4206: fixed a status code check in lsa_LookupNames2 that could cause a segv

diff --git a/source/rpc_server/lsa/dcesrv_lsa.c b/source/rpc_server/lsa/dcesrv_lsa.c
index 22e108d5380f2920354b6cb8594c15ee4b3fc8a5..5b1323dcff8c50b82446c79ab16ed8e68e997e06 100644 (file)
--- a/source/rpc_server/lsa/dcesrv_lsa.c
+++ b/source/rpc_server/lsa/dcesrv_lsa.c
@@ -691,8 +691,9 @@ static NTSTATUS lsa_OpenAccount(struct dcesrv_call_state *dce_call, TALLOC_CTX *
 /* 
   lsa_EnumPrivsAccount 
 */
 /* 
   lsa_EnumPrivsAccount 
 */

-static NTSTATUS lsa_EnumPrivsAccount(struct dcesrv_call_state *dce_call, TALLOC_CTX *mem_ctx,
-                      struct lsa_EnumPrivsAccount *r)
+static NTSTATUS lsa_EnumPrivsAccount(struct dcesrv_call_state *dce_call, 
+                                    TALLOC_CTX *mem_ctx,
+                                    struct lsa_EnumPrivsAccount *r)

 {
        DCESRV_FAULT(DCERPC_FAULT_OP_RNG_ERROR);
 }
 {
        DCESRV_FAULT(DCERPC_FAULT_OP_RNG_ERROR);
 }


@@ -1375,7 +1376,7 @@ static NTSTATUS lsa_LookupNames2(struct dcesrv_call_state *dce_call,
                r->out.sids->sids[i].unknown     = 0;
 
                status2 = lsa_lookup_name(state, mem_ctx, name, &sid, &atype);
                r->out.sids->sids[i].unknown     = 0;
 
                status2 = lsa_lookup_name(state, mem_ctx, name, &sid, &atype);

-               if (!NT_STATUS_IS_OK(status) || sid->num_auths == 0) {
+               if (!NT_STATUS_IS_OK(status2) || sid->num_auths == 0) {

                        status = STATUS_SOME_UNMAPPED;
                        continue;
                }
                        status = STATUS_SOME_UNMAPPED;
                        continue;
                }