OpenAFS Master Repository branch, master, updated. BP--openafs-stable-1_6_x-1534-g10d2734
Gerrit Code Review
gerrit@openafs.org
Sat, 13 Aug 2011 19:09:28 -0700 (PDT)
The following commit has been merged in the master branch:
commit 10d27341808be41b29fbcc09b8bd7523c3b7a541
Author: Garrett Wollman <wollman@csail.mit.edu>
Date: Sat Aug 13 18:51:02 2011 -0400
libafs: don't call afs_PutDCache(NULL) in afs_GetDownD()
It's possible for an eviction candidate to be omitted by the small
for loop (around line 670), leaving its reference in victimDCs
set to NULL. In the big for loop that follows, don't call
afs_PutDCache() when we hit one.
Change-Id: Ib0891636a3479bf97cdeab823189e659cb261aa6
Found-by: clang static analyzer with help from AFS_NONNULL
Reviewed-on: http://gerrit.openafs.org/5260
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Derrick Brashear <shadow@dementix.org>
src/afs/afs_dcache.c | 3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
--
OpenAFS Master Repository