Refactor port binding, warning not yet tested fully

[user/henk/code/inspircd.git] / src / cmd_topic.cpp

diff --git a/src/cmd_topic.cpp b/src/cmd_topic.cpp
index 6d2f6cf9e38b303408de09f0b1b7488310c23355..dfd03a65183884f9db9a9a1ba8e846053a5cf7a2 100644 (file)
--- a/src/cmd_topic.cpp
+++ b/src/cmd_topic.cpp
@@ -2,12 +2,9 @@
  *       | Inspire Internet Relay Chat Daemon |
  *       +------------------------------------+
  *
- *  InspIRCd is copyright (C) 2002-2006 ChatSpike-Dev.
- *                       E-mail:
- *                <brain@chatspike.net>
- *                <Craig@chatspike.net>
+ *  InspIRCd: (C) 2002-2007 InspIRCd Development Team
+ * See: http://www.inspircd.org/wiki/index.php/Credits
  *
- * Written by Craig Edwards, Craig McLure, and others.
  * This program is free but copyrighted software; see
  *            the file COPYING for details.
  *
@@ -71,29 +68,44 @@ CmdResult cmd_topic::Handle (const char** parameters, int pcnt, userrec *user)
                                }
                                if ((Ptr->modes[CM_TOPICLOCK]) && (Ptr->GetStatus(user) < STATUS_HOP))
                                {
-                                       user->WriteServ("482 %s %s :You must be at least a half-operator to change modes on this channel", user->nick, Ptr->name);
+                                       user->WriteServ("482 %s %s :You must be at least a half-operator to change the topic on this channel", user->nick, Ptr->name);
                                        return CMD_FAILURE;
                                }
                        }
+
                        char topic[MAXTOPIC];
-                       strlcpy(topic,parameters[1],MAXTOPIC-1);
 
                        if (IS_LOCAL(user))
                        {
+                               /* XXX: we need two string copies for a local topic, because we cant
+                                * let a module see the topic as longer than it actually is
+                                */
                                int MOD_RESULT = 0;
+
+                               strlcpy(topic,parameters[1],MAXTOPIC-1);
                                FOREACH_RESULT(I_OnLocalTopicChange,OnLocalTopicChange(user,Ptr,topic));
                                if (MOD_RESULT)
                                        return CMD_FAILURE;
+
+                               strlcpy(Ptr->topic,topic,MAXTOPIC-1);
+                       }
+                       else
+                       {
+                               /* Sneaky shortcut, one string copy for a remote topic */
+                               strlcpy(Ptr->topic, parameters[1], MAXTOPIC-1);
                        }
 
-                       strlcpy(Ptr->topic,topic,MAXTOPIC-1);
-                       strlcpy(Ptr->setby,user->nick,NICKMAX-1);
+                       if (ServerInstance->Config->FullHostInTopic)
+                               strlcpy(Ptr->setby,user->GetFullHost(),127);
+                       else
+                               strlcpy(Ptr->setby,user->nick,127);
+
                        Ptr->topicset = ServerInstance->Time();
                        Ptr->WriteChannel(user, "TOPIC %s :%s", Ptr->name, Ptr->topic);
+
                        if (IS_LOCAL(user))
-                       {
-                               FOREACH_MOD(I_OnPostLocalTopicChange,OnPostLocalTopicChange(user,Ptr,topic));
-                       }
+                               /* We know 'topic' will contain valid data here */
+                               FOREACH_MOD(I_OnPostLocalTopicChange,OnPostLocalTopicChange(user, Ptr, topic));
                }
                else
                {