The following commit has been merged in the master branch:
commit a5c146116faccc9891ae6d50506cd43d52dadb70
Author: Michael Meffie <[email protected]>
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 <[email protected]>
    Reviewed-by: Mark Vitale <[email protected]>
    Reviewed-by: Benjamin Kaduk <[email protected]>

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

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

Reply via email to