OpenAFS Master Repository branch, openafs-stable-1_6_x, updated. openafs-stable-1_6_0pre7-44-g3fd135d

Gerrit Code Review gerrit@openafs.org
Fri, 12 Aug 2011 08:52:51 -0700 (PDT)


The following commit has been merged in the openafs-stable-1_6_x branch:
commit 3fd135d48b76b2e54329b75e53a67cac1abb29d4
Author: Garrett Wollman <wollman@csail.mit.edu>
Date:   Sat Aug 6 23:15:14 2011 -0400

    vos: don't free stack garbage on error
    
    If wantExtendedInfo is true, then pntr is used uninitialized.
    In the other case, UV_ListVolumes will have set it to NULL
    before doing anything (even if it returns an error), so this
    free() is dead anyway.
    
    Found-by: clang static analyzer
    Reviewed-on: http://gerrit.openafs.org/5164
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Simon Wilkinson <sxw@inf.ed.ac.uk>
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    (cherry picked from commit 619b420903c99e48618db0d2f12085111573f279)
    
    Change-Id: Icd76d3ac160ccb04b05445552a9e6d04f053efbb
    Reviewed-on: http://gerrit.openafs.org/5227
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Tested-by: Derrick Brashear <shadow@dementia.org>

 src/volser/vos.c |    2 --
 1 files changed, 0 insertions(+), 2 deletions(-)

-- 
OpenAFS Master Repository