summaryrefslogtreecommitdiff
path: root/src/modules/m_remove.cpp
diff options
context:
space:
mode:
authorPeter Powell <petpow@saberuk.com>2017-12-22 12:12:36 +0000
committerGitHub <noreply@github.com>2017-12-22 12:12:36 +0000
commitc8f515121fbdf3e4de693712ef2311cece45477d (patch)
treed92941491ef510a9ec8ff317a9002429ae96a33e /src/modules/m_remove.cpp
parente467fd0a6fc9ba97b2e2f31e654803a86bbb307f (diff)
parent026c579e4cac7d4545b3c8c3a0d690c8509dc713 (diff)
Merge pull request #1446 from B00mX0r/master+wrongnumeric
Fixed misc. instances of ERR_NOSUCHNICK instead of channel numerics Closes #1122.
Diffstat (limited to 'src/modules/m_remove.cpp')
-rw-r--r--src/modules/m_remove.cpp9
1 files changed, 7 insertions, 2 deletions
diff --git a/src/modules/m_remove.cpp b/src/modules/m_remove.cpp
index dfe624702..0d816cc41 100644
--- a/src/modules/m_remove.cpp
+++ b/src/modules/m_remove.cpp
@@ -74,9 +74,14 @@ class RemoveBase : public Command
channel = ServerInstance->FindChan(channame);
/* Fix by brain - someone needs to learn to validate their input! */
- if ((!target) || (target->registered != REG_ALL) || (!channel))
+ if (!channel)
{
- user->WriteNumeric(Numerics::NoSuchNick(channel ? username.c_str() : channame.c_str()));
+ user->WriteNumeric(Numerics::NoSuchChannel(channame));
+ return CMD_FAILURE;
+ }
+ if ((!target) || (target->registered != REG_ALL))
+ {
+ user->WriteNumeric(Numerics::NoSuchNick(username));
return CMD_FAILURE;
}