diff options
author | jhibbits <jhibbits@FreeBSD.org> | 2013-11-20 01:42:29 +0000 |
---|---|---|
committer | jhibbits <jhibbits@FreeBSD.org> | 2013-11-20 01:42:29 +0000 |
commit | eda0b2bf2591ee6dbaaafeb8768e57118ef94ce8 (patch) | |
tree | 6cac87e3535bd1ea2a68f867df32cce44e69719e /cddl | |
parent | 008d8ce40f70c22343cb9fa0c0a1412aa32b9d3f (diff) | |
download | FreeBSD-src-eda0b2bf2591ee6dbaaafeb8768e57118ef94ce8.zip FreeBSD-src-eda0b2bf2591ee6dbaaafeb8768e57118ef94ce8.tar.gz |
Use 'int' to store the return value of getopt(), rather than char.
On some architectures (powerpc), char is unsigned by default, which means
comparisons against -1 always fail, so the programs get stuck in an
infinite loop.
MFC after: 1 week
Diffstat (limited to 'cddl')
-rw-r--r-- | cddl/contrib/opensolaris/cmd/plockstat/plockstat.c | 3 | ||||
-rw-r--r-- | cddl/contrib/opensolaris/cmd/zfs/zfs_main.c | 4 |
2 files changed, 4 insertions, 3 deletions
diff --git a/cddl/contrib/opensolaris/cmd/plockstat/plockstat.c b/cddl/contrib/opensolaris/cmd/plockstat/plockstat.c index aa2c1f9..1288468 100644 --- a/cddl/contrib/opensolaris/cmd/plockstat/plockstat.c +++ b/cddl/contrib/opensolaris/cmd/plockstat/plockstat.c @@ -778,7 +778,8 @@ main(int argc, char **argv) #endif int err; int opt_C = 0, opt_H = 0, opt_p = 0, opt_v = 0; - char c, *p, *end; + int c; + char *p, *end; struct sigaction act; int done = 0; diff --git a/cddl/contrib/opensolaris/cmd/zfs/zfs_main.c b/cddl/contrib/opensolaris/cmd/zfs/zfs_main.c index 6fd8950..81e6e8c 100644 --- a/cddl/contrib/opensolaris/cmd/zfs/zfs_main.c +++ b/cddl/contrib/opensolaris/cmd/zfs/zfs_main.c @@ -2011,7 +2011,7 @@ zfs_do_upgrade(int argc, char **argv) boolean_t showversions = B_FALSE; int ret = 0; upgrade_cbdata_t cb = { 0 }; - char c; + int c; int flags = ZFS_ITER_ARGS_CAN_BE_PATHS; /* check options */ @@ -3561,7 +3561,7 @@ static int zfs_do_snapshot(int argc, char **argv) { int ret = 0; - char c; + int c; nvlist_t *props; snap_cbdata_t sd = { 0 }; boolean_t multiple_snaps = B_FALSE; |