OpenAFS Master Repository branch, master, updated. BP-openafs-stable-1_8_x-266-g0548ee4

Gerrit Code Review gerrit@openafs.org
Fri, 12 Oct 2018 08:47:29 -0400


The following commit has been merged in the master branch:
commit 0548ee436d0f0f92a980d22e03149faedf38dc70
Author: Andrew Deason <adeason@sinenomine.net>
Date:   Mon Oct 1 11:56:53 2018 -0400

    afs: Free 'addrs' array
    
    Currently, 3 places in libafs allocate an 'addrs' array in a very
    similar way to loop through our list of servers:
    ForceAllNewConnections(), afs_LoopServers(), and PCallBackAddr(). Of
    these, only afs_LoopServers actually frees the array.
    ForceAllNewConnections and PCallBackAddr leak the memory, but these
    are only hit from infrequent pioctls that can only be run by root, so
    the impact is small.
    
    Fix ForceAllNewConnections and PCallBackAddr to free the array.
    
    Change-Id: Ic348e29cefa7c41cbcb30f738f943e8d022a97f0
    Reviewed-on: https://gerrit.openafs.org/13355
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>

 src/afs/afs_pioctl.c |    1 +
 src/afs/afs_server.c |    2 ++
 2 files changed, 3 insertions(+), 0 deletions(-)

-- 
OpenAFS Master Repository