From cfca8a451bb13b88ed36a27acfe6cc19a477eef7 Mon Sep 17 00:00:00 2001 From: Martin George Date: Sun, 30 Jun 2024 13:09:07 +0530 Subject: [PATCH] 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 --- nvme.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/nvme.c b/nvme.c index 16dd57b5..91012faf 100644 --- 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); -- 2.50.1