summaryrefslogtreecommitdiff
path: root/source3
diff options
context:
space:
mode:
authorVolker Lendecke <vlendec@samba.org>2007-08-04 10:18:33 +0000
committerGerald (Jerry) Carter <jerry@samba.org>2007-10-10 12:29:17 -0500
commitd7b7ef4a3375cd3302fbab4c208e3dddc4e4a037 (patch)
tree187f031c49abe8eb94637e72f63c3065abf4b33a /source3
parent01ee1c7909bc6840ada40421b9ca1c3b4cbf99ae (diff)
downloadsamba-d7b7ef4a3375cd3302fbab4c208e3dddc4e4a037.tar.gz
samba-d7b7ef4a3375cd3302fbab4c208e3dddc4e4a037.tar.bz2
samba-d7b7ef4a3375cd3302fbab4c208e3dddc4e4a037.zip
r24165: Fix Coverity ID 369.
This was not really a bug I think, but this change cleans up the code a bit. (This used to be commit 59b4914df3ecc97a1c629e8c7a066aed8e8d9226)
Diffstat (limited to 'source3')
-rw-r--r--source3/rpc_server/srv_lsa_nt.c10
1 files changed, 5 insertions, 5 deletions
diff --git a/source3/rpc_server/srv_lsa_nt.c b/source3/rpc_server/srv_lsa_nt.c
index 1f9c245723..40d5005c4f 100644
--- a/source3/rpc_server/srv_lsa_nt.c
+++ b/source3/rpc_server/srv_lsa_nt.c
@@ -1517,16 +1517,16 @@ NTSTATUS _lsa_enum_accounts(pipes_struct *p, LSA_Q_ENUM_ACCOUNTS *q_u, LSA_R_ENU
SAFE_FREE(sid_list);
return NT_STATUS_NO_MEMORY;
}
+
+ for (i = q_u->enum_context, j = 0; i < num_entries; i++, j++) {
+ init_dom_sid2(&(*sids).sid[j], &sid_list[i]);
+ (*sids).ptr_sid[j] = 1;
+ }
} else {
sids->ptr_sid = NULL;
sids->sid = NULL;
}
- for (i = q_u->enum_context, j = 0; i < num_entries; i++, j++) {
- init_dom_sid2(&(*sids).sid[j], &sid_list[i]);
- (*sids).ptr_sid[j] = 1;
- }
-
talloc_free(sid_list);
init_lsa_r_enum_accounts(r_u, num_entries);