diff options
| author | Mike Frysinger <vapier@gentoo.org> | 2010-01-04 23:15:32 -0500 |
|---|---|---|
| committer | Theodore Ts'o <tytso@mit.edu> | 2010-07-05 14:53:58 -0400 |
| commit | b887f08f959fa3ef4ead9b6e2fd71becf4877a40 (patch) | |
| tree | d941568123102dad85b1ffb32127eaf5b4d4ba8b /e2fsck | |
| parent | f96cb89e7aa1ff0d2c4f90b2cf499151f96f10c4 (diff) | |
| download | e2fsprogs-b887f08f959fa3ef4ead9b6e2fd71becf4877a40.tar.gz | |
e2freefrag: Fix getopt bug on machines with unsigned chars
The getopt() function returns an int, not a char. On systems where the
default char is unsigned (like ppc), we get weird behavior where -1 is
truncated to 0xff but compared to (int)-1.
Also fix this same bug for two test programs, test_rel and iscan,
which aren't currently used at the moment.
Addresses-Gentoo-Bug: #299386
Signed-off-by: Mike Frysinger <vapier@gentoo.org>
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
Diffstat (limited to 'e2fsck')
| -rw-r--r-- | e2fsck/iscan.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/e2fsck/iscan.c b/e2fsck/iscan.c index e9fa0763..84e2cc17 100644 --- a/e2fsck/iscan.c +++ b/e2fsck/iscan.c @@ -51,7 +51,7 @@ static void usage(void) static void PRS(int argc, char *argv[]) { int flush = 0; - char c; + int c; #ifdef MTRACE extern void *mallwatch; #endif |