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