summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorC. McEnroe2020-04-05 13:17:38 -0400
committerC. McEnroe2020-04-05 13:17:38 -0400
commit1969c9125fbde0d420c22cebc7578b4473a7aac0 (patch)
tree63d81583acb6f6877aaf3a41f9de9949717d1ec6
parentbb531a71b14a43923d8c8f8cd44564b655aa7dee (diff)
Error on missing MODE parameters
-rw-r--r--handle.c16
1 files changed, 12 insertions, 4 deletions
diff --git a/handle.c b/handle.c
index 8e91810..0719754 100644
--- a/handle.c
+++ b/handle.c
@@ -677,7 +677,9 @@ static void handleMode(struct Message *msg) {
if (!name) name = "";
if (strchr(network.prefixModes, *ch)) {
- assert(i < ParamCap);
+ if (i >= ParamCap || !msg->params[i]) {
+ errx(EX_PROTOCOL, "MODE missing %s parameter", mode);
+ }
char *nick = msg->params[i++];
char prefix = network.prefixes[
strchr(network.prefixModes, *ch) - network.prefixModes
@@ -696,7 +698,9 @@ static void handleMode(struct Message *msg) {
}
if (strchr(network.listModes, *ch)) {
- assert(i < ParamCap);
+ if (i >= ParamCap || !msg->params[i]) {
+ errx(EX_PROTOCOL, "MODE missing %s parameter", mode);
+ }
char *mask = msg->params[i++];
if (*ch == 'b') {
verb = (set ? "bans" : "unbans");
@@ -727,7 +731,9 @@ static void handleMode(struct Message *msg) {
}
if (strchr(network.paramModes, *ch)) {
- assert(i < ParamCap);
+ if (i >= ParamCap || !msg->params[i]) {
+ errx(EX_PROTOCOL, "MODE missing %s parameter", mode);
+ }
char *param = msg->params[i++];
uiFormat(
id, Cold, tagTime(msg),
@@ -742,7 +748,9 @@ static void handleMode(struct Message *msg) {
}
if (strchr(network.setParamModes, *ch) && set) {
- assert(i < ParamCap);
+ if (i >= ParamCap || !msg->params[i]) {
+ errx(EX_PROTOCOL, "MODE missing %s parameter", mode);
+ }
char *param = msg->params[i++];
uiFormat(
id, Cold, tagTime(msg),