oysteikt/mpd
0
0
Fork 0
Code Issues Pull Requests Releases Activity

command/all: fix off-by-one bug in parameter count check

Browse Source
This commit is contained in:
Max Kellermann 2022-08-09 12:47:56 +02:00
parent d0909adf6b
commit 938054bdb8
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/command/AllCommands.cxx
View File

@ -409,15 +409,15 @@ command_process(Client &client, unsigned num, char *line) noexcept
StaticVector<const char *, COMMAND_ARGV_MAX> argv; StaticVector<const char *, COMMAND_ARGV_MAX> argv;
while (true) { while (true) {
char *a = tokenizer.NextParam();
if (a == nullptr)
break;
if (argv.full()) { if (argv.full()) {
r.Error(ACK_ERROR_ARG, "Too many arguments"); r.Error(ACK_ERROR_ARG, "Too many arguments");
return CommandResult::ERROR; return CommandResult::ERROR;
} }
char *a = tokenizer.NextParam();
if (a == nullptr)
break;
argv.push_back(a); argv.push_back(a);
} }