From 3eb205218a321e454d873ae14e2e717ce9d64142 Mon Sep 17 00:00:00 2001 From: Attila Molnar Date: Wed, 16 Apr 2014 13:08:44 +0200 Subject: m_spanningtree Throw an exception on protocol violations instead of returning CMD_INVALID Catch CoreExceptions, log and close the link in OnDataReady() --- src/modules/m_spanningtree/nick.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/modules/m_spanningtree/nick.cpp') diff --git a/src/modules/m_spanningtree/nick.cpp b/src/modules/m_spanningtree/nick.cpp index 49ce9a767..733901632 100644 --- a/src/modules/m_spanningtree/nick.cpp +++ b/src/modules/m_spanningtree/nick.cpp @@ -33,7 +33,7 @@ CmdResult CommandNick::HandleRemote(RemoteUser* user, std::vector& params) { if ((isdigit(params[0][0])) && (params[0] != user->uuid)) - return CMD_INVALID; + throw ProtocolException("Attempted to change nick to an invalid or non-matching UUID"); /* Update timestamp on user when they change nicks */ const time_t newts = ConvToInt(params[1]); -- cgit v1.3.1-10-gc9f91