OpenAFS Master Repository branch, openafs-devel-1_7_x, updated. openafs-devel-1_7_32-26-gaaec2f8

Gerrit Code Review gerrit@openafs.org
Thu, 8 Oct 2015 07:35:13 -0400


The following commit has been merged in the openafs-devel-1_7_x branch:
commit aaec2f89e88c315e836a86bba45324903d24bc4f
Author: Jeffrey Altman <jaltman@your-file-system.com>
Date:   Wed Sep 30 13:23:36 2015 -0400

    Windows: multi ping do not leak ping count
    
    In cm_CheckServersMulti() if cm_ConnByServer() fails or if cm_noIPAddr is
    zero then a cm_server.pingCount will be leaked.  This can result in
    servers being marked down and never restored to an up state.
    
    This change adds the necessary pingCount decrement and moves the
    assignment of the cm_server_t pointer to serversp[] to make it clear
    that the cm_server_t will not be in the array if a failure occurs.
    Only objects in the array will have the pingCount decremented after
    the RPCs are issued.
    
    Reviewed-on: http://gerrit.openafs.org/12048
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
    (cherry picked from commit 3957abea2abcd791e97e66be3f3ef211672ce713)
    
    Change-Id: I4904d280b3a36ad3793fc0aa9728731fc9ac258b
    Reviewed-on: http://gerrit.openafs.org/12052
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>

 src/WINNT/afsd/cm_server.c |   24 ++++++++++++------------
 1 files changed, 12 insertions(+), 12 deletions(-)

-- 
OpenAFS Master Repository