OpenAFS Master Repository branch, master, updated. openafs-devel-1_5_74_1-243-g6dd07af

Gerrit Code Review gerrit@openafs.org
Sat, 12 Jun 2010 22:41:25 -0700 (PDT)


The following commit has been merged in the master branch:
commit 6dd07af215e2cc75cb7c902b5a06920a5ee5235c
Author: Simon Wilkinson <sxw@inf.ed.ac.uk>
Date:   Sat Jun 12 10:26:09 2010 +0100

    viced: CopyOnWrite2 shouldn't return undefined val
    
    Unless CopyOnWrite2 encounters the condition where
    (rdlen != length) || (wrlen !=length) it will never set a value
    for the 'rc' return code, and so return with an undefined value.
    
    Looking at the code, it looks like rc should be 0 in all other
    situations, so this patch provides a default value.
    
    Caught by clang-analyzer
    
    Change-Id: If3e528f3d66abb3dde6da2304084f7698c05b863
    Reviewed-on: http://gerrit.openafs.org/2141
    Reviewed-by: Russ Allbery <rra@stanford.edu>
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Tested-by: Derrick Brashear <shadow@dementia.org>

 src/viced/afsfileprocs.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

-- 
OpenAFS Master Repository