The following commit has been merged in the master branch:
commit c1c4e308cfe0b189778259840f02183c83c1083e
Author: Benjamin Kaduk <[email protected]>
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 <[email protected]>
    Reviewed-by: Benjamin Kaduk <[email protected]>
    Tested-by: BuildBot <[email protected]>

 src/ptserver/ptprocs.c |   35 ++++++++++++-----------------------
 1 files changed, 12 insertions(+), 23 deletions(-)

-- 
OpenAFS Master Repository
_______________________________________________
OpenAFS-cvs mailing list
[email protected]
https://lists.openafs.org/mailman/listinfo/openafs-cvs

Reply via email to