From 05e0dfbb8042ffa00cd71a666794c053d976d7b2 Mon Sep 17 00:00:00 2001 From: Volker Lendecke Date: Tue, 21 Apr 2020 14:24:48 +0200 Subject: srvsvc: Directly use "ctr3->count" instead of "i" To me this was not very transparent, and now that we have "ctr3" a single indirect looks okay Bug: https://bugzilla.samba.org/show_bug.cgi?id=14355 Signed-off-by: Volker Lendecke Reviewed-by: Stefan Metzmacher (cherry picked from commit 96d68bb9f26a0c99d00e92130a2f2c91c7b985e2) --- source3/rpc_server/srvsvc/srv_srvsvc_nt.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/source3/rpc_server/srvsvc/srv_srvsvc_nt.c b/source3/rpc_server/srvsvc/srv_srvsvc_nt.c index b81de3d2b9e..a24ea638624 100644 --- a/source3/rpc_server/srvsvc/srv_srvsvc_nt.c +++ b/source3/rpc_server/srvsvc/srv_srvsvc_nt.c @@ -89,7 +89,6 @@ static int enum_file_fn(struct file_id id, (struct file_enum_count *)private_data; struct srvsvc_NetFileCtr3 *ctr3 = fenum->ctr3; struct srvsvc_NetFileInfo3 *f; - int i = ctr3->count; files_struct fsp; struct byte_range_lock *brl; int num_locks = 0; @@ -114,9 +113,9 @@ static int enum_file_fn(struct file_id id, fenum->ctx, ctr3->array, struct srvsvc_NetFileInfo3, - i+1); + ctr3->count+1); if ( !f ) { - DEBUG(0,("conn_enum_fn: realloc failed for %d items\n", i+1)); + DBG_ERR("realloc failed for %"PRIu32" items\n", ctr3->count+1); return 0; } ctr3->array = f; @@ -154,7 +153,7 @@ static int enum_file_fn(struct file_id id, /* now fill in the srvsvc_NetFileInfo3 struct */ - ctr3->array[i] = (struct srvsvc_NetFileInfo3) { + ctr3->array[ctr3->count] = (struct srvsvc_NetFileInfo3) { .fid = (((uint32_t)(procid_to_pid(&e->pid))<<16) | e->share_file_id), .permissions = permissions, -- cgit v1.2.1