OpenAFS Master Repository branch, openafs-devel-1_7_x, updated. openafs-devel-1_7_11-12-g8c3e63e
Gerrit Code Review
gerrit@openafs.org
Fri, 4 May 2012 21:28:51 -0700 (PDT)
The following commit has been merged in the openafs-devel-1_7_x branch:
commit 8c3e63ef2bc14fb1fe7ca6b3ca940515b68aa504
Author: Jeffrey Altman <jaltman@your-file-system.com>
Date: Sun Apr 29 10:31:07 2012 -0400
Windows: reorg RDR_CleanupFile to prevent lock leak
RDR_CleanupFile could fail to drop a file lock if the user does
not have write permission on last handle close even if the file
is readonly or there were no dirty extents to be stored. The
error handling would return the error immediately and skip the
file lock release. This patchset changes the logic so that the
user permissions are not tested if the file is located on a readonly
volume or if there are no dirty extents or metadata changes to store.
In addition, if there is an error, skip to unlock processing and
not to function exit processing.
Reviewed-on: http://gerrit.openafs.org/7292
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Jeffrey Altman <jaltman@secure-endpoints.com>
Tested-by: Jeffrey Altman <jaltman@secure-endpoints.com>
(cherry picked from commit 50752e2f60d613fda1e70b798a72679bea5d65f8)
Change-Id: I5a5eb35451749f84f6b1ba9e959f27a5eb204c79
Reviewed-on: http://gerrit.openafs.org/7331
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Jeffrey Altman <jaltman@secure-endpoints.com>
src/WINNT/afsrdr/user/RDRFunction.c | 173 ++++++++++++++++++-----------------
1 files changed, 89 insertions(+), 84 deletions(-)
--
OpenAFS Master Repository