/*
* ngIRCd -- The Next Generation IRC Daemon
- * Copyright (c)2001-2014 Alexander Barton (alex@barton.de) and Contributors.
+ * Copyright (c)2001-2023 Alexander Barton (alex@barton.de) and Contributors.
*
* This program is free software; you can redistribute it and/or modify
* it under the terms of the GNU General Public License as published by
Channel_Name(Channel));
break;
}
- if (arg_arg > mode_arg) {
- if (is_oper || is_machine || is_owner ||
- is_admin || is_op || is_halfop) {
- Channel_ModeDel(Channel, 'k');
- Channel_SetKey(Channel,
- Req->argv[arg_arg]);
- strlcpy(argadd, Channel_Key(Channel),
- sizeof(argadd));
- x[0] = *mode_ptr;
- } else {
+ if (arg_arg <= mode_arg) {
+ if (is_machine)
+ Log(LOG_ERR,
+ "Got MODE +k without key for \"%s\" from \"%s\"! Ignored.",
+ Channel_Name(Channel), Client_ID(Origin));
+ else
connected = IRC_WriteErrClient(Origin,
- ERR_CHANOPRIVSNEEDED_MSG,
+ ERR_NEEDMOREPARAMS_MSG,
+ Client_ID(Origin), Req->command);
+ goto chan_exit;
+ }
+ if (!Req->argv[arg_arg][0] || strchr(Req->argv[arg_arg], ' ')) {
+ if (is_machine)
+ Log(LOG_ERR,
+ "Got invalid key on MODE +k for \"%s\" from \"%s\"! Ignored.",
+ Channel_Name(Channel), Client_ID(Origin));
+ else
+ connected = IRC_WriteErrClient(Origin,
+ ERR_INVALIDMODEPARAM_MSG,
Client_ID(Origin),
- Channel_Name(Channel));
- }
- Req->argv[arg_arg][0] = '\0';
- arg_arg++;
+ Channel_Name(Channel), 'k');
+ goto chan_exit;
+ }
+ if (is_oper || is_machine || is_owner ||
+ is_admin || is_op || is_halfop) {
+ Channel_ModeDel(Channel, 'k');
+ Channel_SetKey(Channel, Req->argv[arg_arg]);
+ strlcpy(argadd, Channel_Key(Channel), sizeof(argadd));
+ x[0] = *mode_ptr;
} else {
-#ifdef STRICT_RFC
- /* Only send error message in "strict" mode,
- * this is how ircd2.11 and others behave ... */
connected = IRC_WriteErrClient(Origin,
- ERR_NEEDMOREPARAMS_MSG,
- Client_ID(Origin), Req->command);
-#endif
- goto chan_exit;
+ ERR_CHANOPRIVSNEEDED_MSG,
+ Client_ID(Origin),
+ Channel_Name(Channel));
}
+ Req->argv[arg_arg][0] = '\0';
+ arg_arg++;
break;
case 'l': /* Member limit */
if (Mode_Limit_Reached(Client, mode_arg_count++))
Channel_Name(Channel));
break;
}
- if (arg_arg > mode_arg) {
- if (is_oper || is_machine || is_owner ||
- is_admin || is_op || is_halfop) {
- l = atol(Req->argv[arg_arg]);
- if (l > 0 && l < 0xFFFF) {
- Channel_ModeDel(Channel, 'l');
- Channel_SetMaxUsers(Channel, l);
- snprintf(argadd, sizeof(argadd),
- "%ld", l);
- x[0] = *mode_ptr;
- }
- } else {
+ if (arg_arg <= mode_arg) {
+ if (is_machine)
+ Log(LOG_ERR,
+ "Got MODE +l without limit for \"%s\" from \"%s\"! Ignored.",
+ Channel_Name(Channel), Client_ID(Origin));
+ else
connected = IRC_WriteErrClient(Origin,
- ERR_CHANOPRIVSNEEDED_MSG,
+ ERR_NEEDMOREPARAMS_MSG,
+ Client_ID(Origin), Req->command);
+ goto chan_exit;
+ }
+ l = atol(Req->argv[arg_arg]);
+ if (l <= 0 || l >= 0xFFFF) {
+ if (is_machine)
+ Log(LOG_ERR,
+ "Got MODE +l with invalid limit for \"%s\" from \"%s\"! Ignored.",
+ Channel_Name(Channel), Client_ID(Origin));
+ else
+ connected = IRC_WriteErrClient(Origin,
+ ERR_INVALIDMODEPARAM_MSG,
Client_ID(Origin),
- Channel_Name(Channel));
- }
- Req->argv[arg_arg][0] = '\0';
- arg_arg++;
+ Channel_Name(Channel), 'l');
+ goto chan_exit;
+ }
+ if (is_oper || is_machine || is_owner ||
+ is_admin || is_op || is_halfop) {
+ Channel_ModeDel(Channel, 'l');
+ Channel_SetMaxUsers(Channel, l);
+ snprintf(argadd, sizeof(argadd), "%ld", l);
+ x[0] = *mode_ptr;
} else {
-#ifdef STRICT_RFC
- /* Only send error message in "strict" mode,
- * this is how ircd2.11 and others behave ... */
connected = IRC_WriteErrClient(Origin,
- ERR_NEEDMOREPARAMS_MSG,
- Client_ID(Origin), Req->command);
-#endif
- goto chan_exit;
+ ERR_CHANOPRIVSNEEDED_MSG,
+ Client_ID(Origin),
+ Channel_Name(Channel));
}
+ Req->argv[arg_arg][0] = '\0';
+ arg_arg++;
break;
case 'O': /* IRC operators only */
if (set) {