X-Git-Url: https://git.proxmox.com/?p=pve-common.git;a=blobdiff_plain;f=src%2FPVE%2FCLIHandler.pm;h=43076783db5ff99d6213c3b55414840eb79f94d2;hp=c3829694b77e46bee13acb7a2904da1664425b8a;hb=918140af72a20b738a13810a1a2acc783fc6c8b5;hpb=7bac844eaab512d8789a9103d0efa774a1bab0d6;ds=sidebyside diff --git a/src/PVE/CLIHandler.pm b/src/PVE/CLIHandler.pm index c382969..4307678 100644 --- a/src/PVE/CLIHandler.pm +++ b/src/PVE/CLIHandler.pm @@ -20,6 +20,12 @@ my $assert_initialized = sub { if !($cmddef && $exename && $cli_handler_class); }; +my $abort = sub { + my ($reason, $cmd) = @_; + print_usage_short (\*STDERR, $reason, $cmd); + exit (-1); +}; + my $expand_command_name = sub { my ($def, $cmd) = @_; @@ -361,14 +367,13 @@ my $handle_cmd = sub { $cmddef->{help} = [ __PACKAGE__, 'help', ['cmd'] ]; - # call verifyapi before setup_environment(), because we do not want to - # execute any real code in this case my $cmd = shift @$args; - if (!$cmd) { - print_usage_short (\*STDERR, "no command specified"); - exit (-1); - } elsif ($cmd eq 'verifyapi') { + $abort->("no command specified") if !$cmd; + + # call verifyapi before setup_environment(), don't execute any real code in + # this case + if ($cmd eq 'verifyapi') { PVE::RESTHandler::validate_method_schemas(); return; } @@ -385,11 +390,7 @@ my $handle_cmd = sub { $cmd = &$expand_command_name($cmddef, $cmd); my ($class, $name, $arg_param, $uri_param, $outsub) = @{$cmddef->{$cmd} || []}; - - if (!$class) { - print_usage_short (\*STDERR, "unknown command '$cmd'"); - exit (-1); - } + $abort->("unknown command '$cmd'") if !$class; my $prefix = "$exename $cmd"; my $res = $class->cli_handler($prefix, $name, \@ARGV, $arg_param, $uri_param, $pwcallback, $stringfilemap);