[PATCH] Improve job/cmd line parsing error logging

[fio.git] / parse.c

diff --git a/parse.c b/parse.c
index e001eef4224f554ec3605c9ed9353ceb19af1395..ae8d0d91e2a0a8bc6748d48bd8c98affe3712a03 100644 (file)
--- a/parse.c
+++ b/parse.c
@@ -93,7 +93,7 @@ void strip_blank_end(char *p)
        *(s + 1) = '\0';
 }
 
-static int check_range_bytes(char *str, unsigned long *val)
+static int check_range_bytes(const char *str, unsigned long *val)
 {
        char suffix;
 
@@ -175,9 +175,12 @@ static int handle_option(struct fio_option *o, const char *ptr, void *data)
                *cp = strdup(ptr);
                break;
        case FIO_OPT_RANGE: {
+               char tmp[128];
                char *p1, *p2;
 
-               p1 = strchr(ptr, '-');
+               strcpy(tmp, ptr);
+
+               p1 = strchr(tmp, '-');
                if (!p1) {
                        ret = 1;
                        break;
@@ -185,7 +188,7 @@ static int handle_option(struct fio_option *o, const char *ptr, void *data)
 
                p2 = p1 + 1;
                *p1 = '\0';
-               p1 = ptr;
+               p1 = tmp;
 
                ret = 1;
                if (!check_range_bytes(p1, &ul1) && !check_range_bytes(p2, &ul2)) {
@@ -251,7 +254,11 @@ int parse_cmd_option(const char *opt, const char *val,
                return 1;
        }
 
-       return handle_option(o, val, data);
+       if (!handle_option(o, val, data))
+               return 0;
+
+       fprintf(stderr, "fio: failed parsing %s=%s\n", opt, val);
+       return 1;
 }
 
 int parse_option(const char *opt, struct fio_option *options, void *data)
@@ -279,5 +286,9 @@ int parse_option(const char *opt, struct fio_option *options, void *data)
                return 1;
        }
 
-       return handle_option(o, post, data);
+       if (!handle_option(o, post, data))
+               return 0;
+
+       fprintf(stderr, "fio: failed parsing %s\n", opt);
+       return 1;
 }