OpenAFS Master Repository branch, master, updated. BP--openafs-stable-1_6_x-893-gce5e263

Gerrit Code Review gerrit@openafs.org
Sat, 19 Mar 2011 19:16:29 -0700 (PDT)


The following commit has been merged in the master branch:
commit ce5e263b488f8cb85662031ee08eea448dab2d27
Author: Andrew Deason <adeason@sinenomine.net>
Date:   Wed Mar 16 14:44:56 2011 -0500

    salvager: Fix volume parsing on 64-bit
    
    When an unsigned long is wider than an afs_uint32, comparing the
    afs_uint32 vid to ULONG_MAX is always going to be false (which the
    compiler can warn us about). Fix this by storing to an unsigned long,
    and converting to a volume id after ensuring that the result is not
    too large.
    
    Change-Id: Ifbd724dabd988bc4b1ba6ee8f3dc7fa1a0afb226
    Reviewed-on: http://gerrit.openafs.org/4244
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Simon Wilkinson <sxw@inf.ed.ac.uk>
    Tested-by: Derrick Brashear <shadow@dementia.org>
    Reviewed-by: Derrick Brashear <shadow@dementia.org>

 src/vol/salvaged.c |    6 ++++--
 src/vol/salvager.c |    6 ++++--
 2 files changed, 8 insertions(+), 4 deletions(-)

-- 
OpenAFS Master Repository