Don't allow a blank fantasy command to continue running.

CyberBotX · CyberBotX · commit c97f2a8a1ebe · 2016-06-09T23:06:34.000-04:00
(This can access invalid vector indicies and potentially segfault.)
diff --git a/diceserv.cpp b/diceserv.cpp
@@ -2191,6 +2191,8 @@ bool DiceServData::PreParse(CommandSource &source, const std::vector<Anope::stri
 	// Set up the dice, chan, and comment strings
 	if (source.c)
 	{
+		if (params.size() < 2)
+			return false;
 		this->chanStr = params[0];
 		this->diceStr = params[1];
 		if (expectedChannelPos != 1)

Original file line number	Diff line number	Diff line change
`@@ -2191,6 +2191,8 @@ bool DiceServData::PreParse(CommandSource &source, const std::vector<Anope::stri`
`2191`	`2191`	`// Set up the dice, chan, and comment strings`
`2192`	`2192`	`if (source.c)`
`2193`	`2193`	`{`
	`2194`	`+ if (params.size() < 2)`
	`2195`	`+ return false;`
`2194`	`2196`	`this->chanStr = params[0];`
`2195`	`2197`	`this->diceStr = params[1];`
`2196`	`2198`	`if (expectedChannelPos != 1)`