OpenAFS Master Repository branch, openafs-stable-1_6_x, updated. openafs-stable-1_6_0pre6-62-g5452fa6

Gerrit Code Review gerrit@openafs.org
Wed, 13 Jul 2011 22:44:24 -0700 (PDT)


The following commit has been merged in the openafs-stable-1_6_x branch:
commit 5452fa62c98b08b5ec3a85d2b7c08585294d4906
Author: Jeffrey Altman <jaltman@your-file-system.com>
Date:   Wed Jul 13 08:15:04 2011 -0400

    Windows: not safe to dereference before locking
    
    Throughout cm_server.c, input parameters to functions that
    are protected by cm_serverLock are dereferenced by assignment
    during variable initialization prior to the cm_serverLock being
    obtained.  As a result there is a race which can result in
    either list corruption or dereferencing freed memory.
    
    Reviewed-on: http://gerrit.openafs.org/4985
    Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
    Tested-by: Jeffrey Altman <jaltman@openafs.org>
    (cherry picked from commit 130155ff3c48f2da2433b359588346b4438d24a2)
    
    Change-Id: I02e83faa889bb55b025253bbd1c51a389434eee4
    Reviewed-on: http://gerrit.openafs.org/5014
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Tested-by: Derrick Brashear <shadow@dementia.org>

 src/WINNT/afsd/cm_server.c |   40 +++++++++++++++++++++++++++-------------
 1 files changed, 27 insertions(+), 13 deletions(-)

-- 
OpenAFS Master Repository