Dont allow keys of length >= 32

[user/henk/code/inspircd.git] / src / modes / cmode_k.cpp

diff --git a/src/modes/cmode_k.cpp b/src/modes/cmode_k.cpp
index 4f95f3ad1baf5de5836d218b8d5686563c3d1e0b..fd5808b739b3e2b3c94a9d32a812fea308795b12 100644 (file)
--- a/src/modes/cmode_k.cpp
+++ b/src/modes/cmode_k.cpp
@@ -18,7 +18,32 @@ ModePair ModeChannelKey::ModeSet(userrec* source, userrec* dest, chanrec* channe
         {
                 return std::make_pair(false, parameter);
         }
-}       
+}
+
+void ModeChannelKey::RemoveMode(chanrec* channel)
+{
+       /** +k needs a parameter when being removed,
+        * so we have a special-case RemoveMode here for it
+        */
+       char moderemove[MAXBUF];
+       const char* parameters[] = { channel->name, moderemove, channel->key };
+
+       if (channel->IsModeSet(this->GetModeChar()))
+       {
+               userrec* n = new userrec(ServerInstance);
+
+               sprintf(moderemove,"-%c",this->GetModeChar());
+               n->SetFd(FD_MAGIC_NUMBER);
+
+               ServerInstance->SendMode(parameters, 3, n);
+
+               delete n;
+       }
+}
+
+void ModeChannelKey::RemoveMode(userrec* user)
+{
+}
 
 bool ModeChannelKey::CheckTimeStamp(time_t theirs, time_t ours, const std::string &their_param, const std::string &our_param, chanrec* channel)
 {
@@ -33,13 +58,12 @@ ModeAction ModeChannelKey::OnModeChange(userrec* source, userrec* dest, chanrec*
                if (((channel->modes[CM_KEY]) && (strcasecmp(parameter.c_str(),channel->key))) && (IS_LOCAL(source)))
                {
                        /* Key is currently set and the correct key wasnt given */
-                       ServerInstance->Log(DEBUG,"Key Cond 2");
                        return MODEACTION_DENY;
                }
                else if ((!channel->modes[CM_KEY]) || ((adding) && (!IS_LOCAL(source))))
                {
                        /* Key isnt currently set */
-                       if ((parameter.length()) && (parameter.rfind(' ') == std::string::npos))
+                       if ((parameter.length()) && (parameter.length() < 32) && (parameter.rfind(' ') == std::string::npos))
                        {
                                strlcpy(channel->key,parameter.c_str(),32);
                                channel->modes[CM_KEY] = adding;
@@ -55,12 +79,11 @@ ModeAction ModeChannelKey::OnModeChange(userrec* source, userrec* dest, chanrec*
                        channel->modes[CM_KEY] = adding;
                        return MODEACTION_ALLOW;
                }
-               ServerInstance->Log(DEBUG,"Key Cond three");
                return MODEACTION_DENY;
        }
        else
        {
-               ServerInstance->Log(DEBUG,"Key Condition one");
                return MODEACTION_DENY;
        }
 }
+