diff options
| author | loli10K <[email protected]> | 2019-05-28 20:14:58 +0200 |
|---|---|---|
| committer | Brian Behlendorf <[email protected]> | 2019-05-28 11:14:58 -0700 |
| commit | 0869b74a1edc1e3375b560f788038615f37feac4 (patch) | |
| tree | af9285b94298a84b942ef22fe109f795c7adea92 /cmd/zstreamdump | |
| parent | 841a7a98fc4623e0bcecfa393e7818ade4198a15 (diff) | |
Endless loop in zpool_do_remove() on platforms with unsigned char
On systems where "char" is an unsigned type the value returned by
getopt() will never be negative (-1), leading to an endless loop:
this issue prevents both 'zpool remove' and 'zstreamdump' for
working on some systems.
Reviewed-by: Igor Kozhukhov <[email protected]>
Reviewed-by: Brian Behlendorf <[email protected]>
Reviewed-by: Chris Dunlop <[email protected]>
Signed-off-by: loli10K <[email protected]>
Closes #8789
Diffstat (limited to 'cmd/zstreamdump')
| -rw-r--r-- | cmd/zstreamdump/zstreamdump.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/cmd/zstreamdump/zstreamdump.c b/cmd/zstreamdump/zstreamdump.c index ed88729b5..a162eceda 100644 --- a/cmd/zstreamdump/zstreamdump.c +++ b/cmd/zstreamdump/zstreamdump.c @@ -237,7 +237,7 @@ main(int argc, char *argv[]) struct drr_write_embedded *drrwe = &thedrr.drr_u.drr_write_embedded; struct drr_object_range *drror = &thedrr.drr_u.drr_object_range; struct drr_checksum *drrc = &thedrr.drr_u.drr_checksum; - char c; + int c; boolean_t verbose = B_FALSE; boolean_t very_verbose = B_FALSE; boolean_t first = B_TRUE; |