]> git.michaelhowe.org Git - packages/o/openafs.git/commit
libafs: Avoid duplicate afs_Analyze in bulk stat
authorAndrew Deason <adeason@sinenomine.net>
Fri, 22 Jul 2011 21:09:52 +0000 (16:09 -0500)
committerDerrick Brashear <shadow@dementix.org>
Sat, 13 Aug 2011 12:56:50 +0000 (05:56 -0700)
commitbf236c0f8a2bb7aa10c776fc90b4fd00e6ada823
tree18287c2af5db2144128778e34cf85781123f0908
parent9b090b9f776835a4d23202e5dbcbca70ff9087df
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