OpenAFS Master Repository branch, openafs-stable-1_6_x, updated. openafs-stable-1_6_0pre7-62-gbf236c0

Gerrit Code Review gerrit@openafs.org
Sat, 13 Aug 2011 05:57:35 -0700 (PDT)


The following commit has been merged in the openafs-stable-1_6_x branch:
commit bf236c0f8a2bb7aa10c776fc90b4fd00e6ada823
Author: Andrew Deason <adeason@sinenomine.net>
Date:   Fri Jul 22 16:09:52 2011 -0500

    libafs: Avoid duplicate afs_Analyze in bulk stat
    
    In afs_DoBulkStat, we can call afs_Analyze multiple times for the same
    set of connection objects. Since afs_Analyze puts its reference to the
    given afs_conn and rx_connection structures, calling it more than once
    can cause the reference counts on those objects to be lower than they
    should be.
    
    Instead of making another afs_Analyze call, just alter the error code
    inside the normal do/while afs_Analyze loop, so the 'loop' afs_Analyze
    call gets the appropriate error code from the first bulk stat'd entry.
    
    (cherry picked from commit ef28bc08c18e750f6100535665d5258a317a0a2b)
    Reviewed-on: http://gerrit.openafs.org/5086
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Tested-by: Derrick Brashear <shadow@dementia.org>
    
    Change-Id: I8bb484aade39830a56cb967a3e3167e63777e333
    Reviewed-on: http://gerrit.openafs.org/5127
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Derrick Brashear <shadow@dementix.org>

 src/afs/VNOPS/afs_vnop_lookup.c |   33 +++++----------------------------
 1 files changed, 5 insertions(+), 28 deletions(-)

-- 
OpenAFS Master Repository