OpenAFS Master Repository branch, openafs-stable-1_4_x, updated. openafs-stable-1_4_12-121-gbc905dd

Gerrit Code Review gerrit@openafs.org
Tue, 7 Jun 2011 09:25:37 -0700 (PDT)


The following commit has been merged in the openafs-stable-1_4_x branch:
commit bc905dd9b3d097211e2d4c64eec1ea55a4ab4d12
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: I40af9d9aad3fa8651c0f16608bdb1bea2ec0f437
    Reviewed-on: http://gerrit.openafs.org/4436
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Derrick Brashear <shadow@dementia.org>

 src/afs/VNOPS/afs_vnop_flock.c |    8 +++++++-
 1 files changed, 7 insertions(+), 1 deletions(-)

-- 
OpenAFS Master Repository