parse: fix minval checking
authorJens Axboe <axboe@kernel.dk>
Wed, 26 Sep 2018 01:44:35 +0000 (19:44 -0600)
committerJens Axboe <axboe@kernel.dk>
Wed, 26 Sep 2018 01:44:35 +0000 (19:44 -0600)
For values that allow negative min values, don't cast to an uint.

Fixes: 5d2788d5b798 ("parse: fix min/max val checking for FIO_OPT_INT")
Signed-off-by: Jens Axboe <axboe@kernel.dk>
parse.c

diff --git a/parse.c b/parse.c
index 9a20d0c..84d112d 100644 (file)
--- a/parse.c
+++ b/parse.c
@@ -512,11 +512,8 @@ static bool val_too_large(const struct fio_option *o, unsigned long long val,
        if (!o->maxval)
                return false;
 
-       if (is_uint) {
-               unsigned int uint_val = val;
-
-               return uint_val > o->maxval;
-       }
+       if (is_uint)
+               return (unsigned int) val > o->maxval;
 
        return val > o->maxval;
 }
@@ -527,11 +524,8 @@ static bool val_too_small(const struct fio_option *o, unsigned long long val,
        if (!o->minval)
                return false;
 
-       if (is_uint) {
-               unsigned int uint_val = val;
-
-               return uint_val < o->minval;
-       }
+       if (is_uint)
+               return (int) val < o->minval;
 
        return val < o->minval;
 }