OpenAFS Master Repository branch, master, updated. openafs-devel-1_9_0-34-gc1c4e30
Gerrit Code Review
gerrit@openafs.org
Mon, 2 Nov 2020 03:13:00 -0500
The following commit has been merged in the master branch:
commit c1c4e308cfe0b189778259840f02183c83c1083e
Author: Benjamin Kaduk <kaduk@mit.edu>
Date: Thu Aug 30 09:54:23 2018 -0500
ptserver: move allocation out of put_prentries() into listEntries()
put_prentries() is a helper function for listEntries(), but the contract
between the two is rather odd -- put_prentries() is expected to notice
when the backing store has not yet been allocated and silently allocate
it, even though there is only the single caller and the allocation could
be done in the caller.
Move the allocation to the caller and adjust the "buffer is full"
logic accordingly, and normalize the initialization of the output
array to just use calloc() instead of individual memset()s when
populating each entry.
Change-Id: Icf84e3b60eae81a1570b12d7adbf006a24a104f3
Reviewed-on: https://gerrit.openafs.org/13315
Reviewed-by: Michael Meffie <mmeffie@sinenomine.net>
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
src/ptserver/ptprocs.c | 35 ++++++++++++-----------------------
1 files changed, 12 insertions(+), 23 deletions(-)
--
OpenAFS Master Repository