nvme: update parse_args() return value handling

Treat non-zero return values from parse_args() as errors, and not
just negative return values. Also makes this consistent with other
functions in nvme.c where parse_args() is invoked.

Signed-off-by: Martin George <marting@netapp.com>

diff --git a/nvme.c b/nvme.c
index 16dd57b586fbf0a0debd9f0ca3e75c9e7d48cf6b..91012faf98e55ceabcc4a09ada9f5581f672dcad 100644 (file)
--- a/nvme.c
+++ b/nvme.c
@@ -3320,7 +3320,7 @@ static int list_subsys(int argc, char **argv, struct command *cmd,
        NVME_ARGS(opts);
 
        err = parse_args(argc, argv, desc, opts);
-       if (err < 0)
+       if (err)
                return err;
 
        devname = NULL;
@@ -3378,7 +3378,7 @@ static int list(int argc, char **argv, struct command *cmd, struct plugin *plugi
        NVME_ARGS(opts);
 
        err = parse_args(argc, argv, desc, opts);
-       if (err < 0)
+       if (err)
                return err;
 
        err = validate_output_format(nvme_cfg.output_format, &flags);