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