From 3c55808e27f552fcfd822a4a2519b59d2d0f3cc1 Mon Sep 17 00:00:00 2001 From: Leah Rowe Date: Sun, 8 Mar 2026 16:27:03 +0000 Subject: util/nvmutil: call usage() on cmd validation err Signed-off-by: Leah Rowe --- util/nvmutil/nvmutil.c | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) (limited to 'util/nvmutil') diff --git a/util/nvmutil/nvmutil.c b/util/nvmutil/nvmutil.c index 534e04cd..f32a170e 100644 --- a/util/nvmutil/nvmutil.c +++ b/util/nvmutil/nvmutil.c @@ -74,7 +74,7 @@ static off_t gbe_x_offset(size_t part, const char *f_op, const char *d_type, off_t nsize, off_t ncmp); static void set_part_modified(size_t p); static void check_part_num(size_t p); -static void usage(void); +static void usage(uint8_t usage_exit); static size_t xstrxlen(const char *scmp, size_t maxlen); static int xstrxcmp(const char *a, const char *b, size_t maxlen); static void err(int nvm_errval, const char *msg, ...); @@ -216,7 +216,6 @@ struct commands { * Pointers used for running nvmutil commands */ static const struct commands command[] = { - { CMD_DUMP, "dump", cmd_dump, ARGC_3, NO_INVERT, SET_MOD_OFF }, @@ -261,7 +260,7 @@ main(int argc, char *argv[]) #endif argv0 = argv[0]; if (argc < 2) - usage(); + usage(1); fname = argv[1]; @@ -483,8 +482,10 @@ set_cmd_args(int argc, char *argv[]) * MAC address is used. */ - if (!valid_command(cmd_index)) + if (!valid_command(cmd_index)) { + usage(0); err(EINVAL, "Unhandled command error"); + } } static size_t @@ -1140,7 +1141,7 @@ check_part_num(size_t p) } static void -usage(void) +usage(uint8_t usage_exit) { const char *util = getnvmprogname(); @@ -1160,7 +1161,8 @@ usage(void) "\t%s FILE setchecksum 0|1\n", util, util, util, util, util, util, util); - err(ECANCELED, "Too few arguments"); + if (usage_exit) + err(ECANCELED, "Too few arguments"); } /* -- cgit v1.2.1