OpenAFS Master Repository branch, master, updated. openafs-devel-1_9_1-51-ga5c1461

Gerrit Code Review gerrit@openafs.org
Sat, 3 Jul 2021 11:37:47 -0400


The following commit has been merged in the master branch:
commit a5c146116faccc9891ae6d50506cd43d52dadb70
Author: Michael Meffie <mmeffie@sinenomine.net>
Date:   Thu Jul 1 20:14:12 2021 -0400

    libadmin: Be sure to free the server list in ListCellsRPC
    
    Commit 0d6b9defb36cb94f3d34b058f00055e9e99d85fc added a check to avoid
    overrunning a buffer when the cell name returned by RXAFSCB_GetCellServDB
    is too large for the libadmin buffer, but in that case missed freeing
    the server list returned in the same rpc.
    
    Move the xdr_free() of the server list under the cleanup label so the
    server list returned by the RPC will always be freed.
    
    Thanks to Mark Vitale for finding this bug.
    
    Change-Id: I4c7228f8c8d48e267f6fbb01cb89fdf5232de97e
    Reviewed-on: https://gerrit.openafs.org/14690
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Mark Vitale <mvitale@sinenomine.net>
    Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>

 src/libadmin/adminutil/afs_utilAdmin.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

-- 
OpenAFS Master Repository