OpenAFS Master Repository branch, master, updated. openafs-devel-1_5_76-3536-g816b0c7
Gerrit Code Review
gerrit@openafs.org
Wed, 27 Feb 2013 12:45:43 -0800 (PST)
The following commit has been merged in the master branch:
commit 816b0c76738b7e404c9384a745b58b4d90bfc30d
Author: Simon Wilkinson <sxw@your-file-system.com>
Date: Wed Feb 27 10:28:05 2013 +0000
Unix CM: Don't free cell, then release lock on it
If afs_NewCell fails, then we can end up releasing a lock on a
section of memory that we have already freed. As this only happens
if the memory we're operating on is newly allocated and not yet
visible to anyone else, it is safe to release the lock before
starting to tidy things up.
Caught by coverity (#986054)
Change-Id: Ie8651c61790d57a9fd7bbbafcaf78e37b8222bae
Reviewed-on: http://gerrit.openafs.org/9298
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
src/afs/afs_cell.c | 6 +++++-
1 files changed, 5 insertions(+), 1 deletions(-)
--
OpenAFS Master Repository