OpenAFS Master Repository branch, master, updated. BP--openafs-stable-1_6_x-859-g0884e9d

Gerrit Code Review gerrit@openafs.org
Thu, 10 Mar 2011 16:25:34 -0800 (PST)


The following commit has been merged in the master branch:
commit 0884e9d0fddf2be81abf6468209048331efa8a1e
Author: Andrew Deason <adeason@sinenomine.net>
Date:   Thu Mar 10 14:55:50 2011 -0600

    salvager: Do not abort on large volume IDs
    
    The salvager was parsing volume IDs just using atoi() and checking if
    the result was negative. Since the result is a signed int, this fails
    on any volume ID larger than 2^31-1. Change the parser to use strtoul
    instead of atoi, and change the check.
    
    While we're here, make a similar change to the DAFS salvageserver,
    too.
    
    Change-Id: Icc3377ee507150ff0c53b5bbff6172cb72bca703
    Reviewed-on: http://gerrit.openafs.org/4196
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Derrick Brashear <shadow@dementia.org>

 src/vol/salvaged.c |   10 ++++++++--
 src/vol/salvager.c |   14 ++++++++------
 2 files changed, 16 insertions(+), 8 deletions(-)

-- 
OpenAFS Master Repository