OpenAFS Master Repository branch, openafs-stable-1_4_x, updated. c08609ae8a65f36179dfcd0ba93938b2369afa4f

Gerrit Code Review gerrit@openafs.org
Mon, 2 Nov 2009 10:52:17 -0800 (PST)


The following commit has been merged in the openafs-stable-1_4_x branch:
commit c08609ae8a65f36179dfcd0ba93938b2369afa4f
Author: Andrew Deason <adeason@sinenomine.net>
Date:   Wed Oct 28 11:06:47 2009 -0500

    Avoid using released hosts
    
    Since h_Release_r has the possibility of freeing a host, we should not
    be using a host after it has been released. A few places can still use a
    released host, potentially causing heap corruption, double frees, and
    generally weird behavior.
    
    So either move calls of h_Release_r until after we finish using a host,
    or make sure to set the pointer to NULL after it has been released.
    
    Reviewed-on: http://gerrit.openafs.org/747
    Tested-by: Andrew Deason <adeason@sinenomine.net>
    Reviewed-by: Dan Hyde <drh@umich.edu>
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    (cherry picked from commit 416e2f11c35f5d55f91090b30b4db1a9bf6d6e07)
    
    Change-Id: I91bd09c3e6d87476de8c66c2eb710c0fd424cadd
    Reviewed-on: http://gerrit.openafs.org/762
    Tested-by: Andrew Deason <adeason@sinenomine.net>
    Reviewed-by: Derrick Brashear <shadow@dementia.org>

 src/viced/afsfileprocs.c |    6 +++---
 src/viced/callback.c     |    4 +++-
 src/viced/host.c         |    9 +++++----
 3 files changed, 11 insertions(+), 8 deletions(-)

-- 
OpenAFS Master Repository