OpenAFS Master Repository branch, master, updated. BP--openafs-stable-1_6_x-694-g5817a13

Gerrit Code Review gerrit@openafs.org
Tue, 25 Jan 2011 12:07:25 -0800 (PST)


The following commit has been merged in the master branch:
commit 5817a13b84496c087b350be50c17a8b0f9aac9d5
Author: Andrew Deason <adeason@sinenomine.net>
Date:   Tue Jan 25 10:46:38 2011 -0600

    vol-salvage: Only delete bad vnodes during !check
    
    In SalvageIndex, if check is true, we're only supposed to check for
    consistency and not actually change anything (this is enforced by an
    assert later on). The check for vnode magic consistency breaks this
    assumption, and just always fixes the vnode.
    
    Change this so we just error out if check is set, and the vnode magic
    is wrong.
    
    Change-Id: I9e2d3bd5e9e340a9b2ac66d70d52aa110b512561
    Reviewed-on: http://gerrit.openafs.org/3756
    Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Tested-by: Derrick Brashear <shadow@dementia.org>

 src/vol/vol-salvage.c |    8 ++++++++
 1 files changed, 8 insertions(+), 0 deletions(-)

-- 
OpenAFS Master Repository