char **cp;
int ret = 0, is_time = 0;
+ if (!ptr && o->type != FIO_OPT_STR_SET) {
+ fprintf(stderr, "Option %s requires an argument\n", o->name);
+ return 1;
+ }
+
switch (o->type) {
case FIO_OPT_STR: {
fio_opt_str_fn *fn = o->cb;
break;
}
default:
- fprintf(stderr, "Bad option type %d\n", o->type);
+ fprintf(stderr, "Bad option type %u\n", o->type);
ret = 1;
}
static int handle_option(struct fio_option *o, const char *ptr, void *data)
{
- const char *ptr2;
+ const char *ptr2 = NULL;
int r1, r2;
/*
* Do this before parsing the first round, to check if we should
* copy set 1 options to set 2.
*/
- ptr2 = strchr(ptr, ',');
+ if (ptr)
+ ptr2 = strchr(ptr, ',');
/*
* Don't return early if parsing the first option fails - if