OpenAFS Master Repository branch, master, updated. openafs-devel-1_5_76-2806-g351d3b8
Gerrit Code Review
gerrit@openafs.org
Tue, 31 Jul 2012 12:45:22 -0700 (PDT)
The following commit has been merged in the master branch:
commit 351d3b8a19314027b30cdc499ef48c95ba7903b6
Author: Andrew Deason <adeason@sinenomine.net>
Date: Tue Jul 31 14:40:41 2012 -0400
LINUX: Always hold afs_xuser for unixuser read
We were failing to hold the afs_xuser lock when we entered our
unixuser traversal for the first time (when the given position is 0).
This means we can release the lock without acquiring it, causing all
kinds of weird behavior.
Just always grab afs_xuser on entry. We could possibly do some tricks
to avoid grabbing this lock until after we've printed the column
headers, but it does not seem worth it.
Change-Id: I68782098b5af2feb56887bc577511da2983d4f21
Reviewed-on: http://gerrit.openafs.org/7916
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Derrick Brashear <shadow@dementix.org>
src/afs/LINUX/osi_proc.c | 8 +++++---
1 files changed, 5 insertions(+), 3 deletions(-)
--
OpenAFS Master Repository