OpenAFS Master Repository branch, openafs-stable-1_6_x, updated. openafs-stable-1_6_0b-111-g741bea0
Gerrit Code Review
gerrit@openafs.org
Tue, 1 Nov 2011 19:44:17 -0700 (PDT)
The following commit has been merged in the openafs-stable-1_6_x branch:
commit 741bea05a3a5d170d47ca9df590f84ab02a7baad
Author: Andrew Deason <adeason@sinenomine.net>
Date: Fri Apr 1 16:43:24 2011 -0500
afs: Avoid memory leak on recursive write flock
When a process requests an exclusive lock on a file on which it
already holds an exclusive lock, we basically form a no-op. However,
HandleFlock was allocating a new SimpleLocks and attaching it to
avc->slocks, without freeing the old SimpleLocks structure.
Since we don't need to do anything if we already hold an exclusive
lock, just break out of the loop right away when we detect that
scenario. Thus we avoid adding a new structure to avc->slocks, and we
avoid a memory leak.
Reviewed-on: http://gerrit.openafs.org/4395
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
(cherry picked from commit ba9ae1ed7c269d7c080b5ce99b3b4bb2fe0a2a6f)
Change-Id: I9e45a51e02774c555c6d5cb5ba9d0407b8442215
Reviewed-on: http://gerrit.openafs.org/5765
Reviewed-by: Derrick Brashear <shadow@dementix.org>
Tested-by: Derrick Brashear <shadow@dementix.org>
src/afs/VNOPS/afs_vnop_flock.c | 8 +++++++-
1 files changed, 7 insertions(+), 1 deletions(-)
--
OpenAFS Master Repository