OpenAFS Master Repository branch, openafs-stable-1_6_x, updated. BP--openafs-stable-1_6_x-5-g96d4012
Gerrit Code Review
gerrit@openafs.org
Tue, 7 Sep 2010 11:44:37 -0700 (PDT)
The following commit has been merged in the openafs-stable-1_6_x branch:
commit 96d40120207e7eea31eed0bb408d2d8abfdaba59
Author: Marc Dionne <marc.c.dionne@gmail.com>
Date: Fri Aug 20 16:37:56 2010 -0400
Fixes for signed char architectures
"char" is unsigned on s390x, which causes a few warnings where we
test for negative values.
Add a signed modifier in a few cases, and in vos.c replace with an
int to match a similar section of code in the same file.
Mostly a warning fix, but in one case the variable could be used
to return a -1 error value, so it may have caused an actual bug.
Change-Id: I005fbcd7c573735df7403316ec0ced67ee81e559
Reviewed-on: http://gerrit.openafs.org/2602
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>
(cherry picked from commit c2be1ee1d726463590ae1272495d102111fdf75b)
Reviewed-on: http://gerrit.openafs.org/2671
src/afs/afs.h | 2 +-
src/rxkad/rxkad.p.h | 2 +-
src/volser/vos.c | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
--
OpenAFS Master Repository