diff options
| author |  danieldg | 2010-01-16 19:32:31 +0000 |
|---|---|---|
| committer | danieldg | 2010-01-16 19:32:31 +0000 |
| commit | f4d23fddb163d71b5f61db85050268c2dbb6d91b (patch) | |
| tree | b1f708c79811771de4513dff5f267d6b2b693d4c /src/command_parse.cpp | |
| parent | Allow maxtargets to be bypassed in LoopCall for JOIN (diff) | |
Make parameter unsigned so that passing -1 for "none" actually works, rather than crashing
git-svn-id: http://svn.inspircd.org/repository/branches/1_2_stable@12266 e03df62e-2008-0410-955e-edbf42e46eb7
Diffstat (limited to 'src/command_parse.cpp')
| -rw-r--r-- | src/command_parse.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/command_parse.cpp b/src/command_parse.cpp index a89156b3a..642dfd35d 100644 --- a/src/command_parse.cpp +++ b/src/command_parse.cpp @@ -57,12 +57,12 @@ int InspIRCd::PassCompare(Extensible* ex, const std::string &data, const std::st * The second version is much simpler and just has the one stream to read, and is used in NAMES, WHOIS, PRIVMSG etc. * Both will only parse until they reach ServerInstance->Config->MaxTargets number of targets, to stop abuse via spam. */ -int CommandParser::LoopCall(User* user, Command* CommandObj, const std::vector<std::string>& parameters, unsigned int splithere, unsigned int extra, bool usemax) +int CommandParser::LoopCall(User* user, Command* CommandObj, const std::vector<std::string>& parameters, unsigned int splithere, int extra, bool usemax) { if (splithere >= parameters.size()) return 0; - if (extra >= parameters.size()) + if (extra >= (signed)parameters.size()) extra = -1; /* First check if we have more than one item in the list, if we don't we return zero here and the handler |