OpenAFS Master Repository branch, master, updated. BP--openafs-stable-1_6_x-1190-g89b22df
Gerrit Code Review
gerrit@openafs.org
Thu, 19 May 2011 05:03:22 -0700 (PDT)
The following commit has been merged in the master branch:
commit 89b22dfe8659cd2e576cc2cd2b455598db59aacc
Author: Andrew Deason <adeason@sinenomine.net>
Date: Thu May 5 15:10:54 2011 -0500
libafs: Implement unixuser RW locks
Currently code dealing with changing unixuser structs does not obtain
any locks protecting the contents of the unixuser struct, though some
functions like afs_GetUser have a parameter indicating what type of
lock should be obtained. This can result in the token data for a user
being changed at the same time another thread tries to use the token
data.
To ensure mutual exclusion of such operations, add a lock field to the
unixuser struct, and actually lock it according to the intentions of
the relevant code.
Change-Id: Idd66d72f716b7e7dc08faa31ae43e9a23639bae3
Reviewed-on: http://gerrit.openafs.org/4636
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>
src/afs/DOC/afs_rwlocks | 4 +++
src/afs/LINUX/osi_proc.c | 8 ++++++
src/afs/LINUX24/osi_proc.c | 8 ++++++
src/afs/VNOPS/afs_vnop_lookup.c | 12 +++++-----
src/afs/afs.h | 1 +
src/afs/afs_nfsclnt.c | 6 +++-
src/afs/afs_pag_cred.c | 24 +++++++++++++++++++-
src/afs/afs_pioctl.c | 25 +++++++++++++++------
src/afs/afs_prototypes.h | 2 +
src/afs/afs_user.c | 46 +++++++++++++++++++++++++++++++++++---
src/afs/afs_util.c | 2 +
11 files changed, 118 insertions(+), 20 deletions(-)
--
OpenAFS Master Repository