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 09565faea51e624306a2ddcbbb898888db987a12
Author: Garrett Wollman <wollman@csail.mit.edu>
Date:   Sat Aug 6 23:49:10 2011 -0400

    butc: avoid testing stack garbage; remove dead initializer
    
    "code" is unconditionally set early in saveDbToTape() so there's
    no need to initialize it.  On the other hand, dumpEntry.id is used
    before dumpEntry is initialized, so set it to what appears to be
    the expected value before any non-local exits could cause it to be
    inspected.
    
    Found-by: clang static analyzer
    Reviewed-on: http://gerrit.openafs.org/5166
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    (cherry picked from commit c92f04af6094ce04b1541a1b23b254d3c1421290)
    
    Change-Id: If99aaf327749550623ef5abc3dd077f5291bc899
    Reviewed-on: http://gerrit.openafs.org/5226
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Tested-by: Derrick Brashear <shadow@dementia.org>

 src/butc/tcudbprocs.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

-- 
OpenAFS Master Repository