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