Fix MySQL crash on module unload with empty query queue

[user/henk/code/inspircd.git] / src / commands / cmd_list.cpp

diff --git a/src/commands/cmd_list.cpp b/src/commands/cmd_list.cpp
index 2ead367a4642503864b2e544bc7067300d2cb01a..b820135f87fcf99d6a9e503d85c708ffcfccedce 100644 (file)
--- a/src/commands/cmd_list.cpp
+++ b/src/commands/cmd_list.cpp
@@ -2,8 +2,8 @@
  *       | Inspire Internet Relay Chat Daemon |
  *       +------------------------------------+
  *
- *  InspIRCd: (C) 2002-2008 InspIRCd Development Team
- * See: http://www.inspircd.org/wiki/index.php/Credits
+ *  InspIRCd: (C) 2002-2010 InspIRCd Development Team
+ * See: http://wiki.inspircd.org/Credits
  *
  * This program is free but copyrighted software; see
  *            the file COPYING for details.
@@ -12,34 +12,46 @@
  */
 
 #include "inspircd.h"
-#include "commands/cmd_list.h"
-#include "wildcard.h"
 
-/** Handle /LIST
+/** Handle /LIST. These command handlers can be reloaded by the core,
+ * and handle basic RFC1459 commands. Commands within modules work
+ * the same way, however, they can be fully unloaded, where these
+ * may not.
  */
-extern "C" DllExport Command* init_command(InspIRCd* Instance)
+class CommandList : public Command
 {
-       return new CommandList(Instance);
-}
+ public:
+       /** Constructor for list.
+        */
+       CommandList ( Module* parent) : Command(parent,"LIST", 0, 0) { Penalty = 5; }
+       /** Handle command.
+        * @param parameters The parameters to the comamnd
+        * @param pcnt The number of parameters passed to teh command
+        * @param user The user issuing the command
+        * @return A value from CmdResult to indicate command success or failure.
+        */
+       CmdResult Handle(const std::vector<std::string>& parameters, User *user);
+};
 
-CmdResult CommandList::Handle (const char* const* parameters, int pcnt, User *user)
+
+/** Handle /LIST
+ */
+CmdResult CommandList::Handle (const std::vector<std::string>& parameters, User *user)
 {
        int minusers = 0, maxusers = 0;
 
-       user->WriteServ("321 %s Channel :Users Name",user->nick);
+       user->WriteNumeric(321, "%s Channel :Users Name",user->nick.c_str());
 
        /* Work around mIRC suckyness. YOU SUCK, KHALED! */
-       if (pcnt == 1)
+       if (parameters.size() == 1)
        {
-               if (*parameters[0] == '<')
+               if (parameters[0][0] == '<')
                {
-                       maxusers = atoi(parameters[0]+1);
-                       pcnt = 0;
+                       maxusers = atoi((parameters[0].c_str())+1);
                }
-               else if (*parameters[0] == '>')
+               else if (parameters[0][0] == '>')
                {
-                       minusers = atoi(parameters[0]+1);
-                       pcnt = 0;
+                       minusers = atoi((parameters[0].c_str())+1);
                }
        }
 
@@ -54,28 +66,33 @@ CmdResult CommandList::Handle (const char* const* parameters, int pcnt, User *us
                if (too_many || too_few)
                        continue;
 
-               if (pcnt)
+               if (parameters.size() && (parameters[0][0] != '<' && parameters[0][0] != '>'))
                {
-                       if (!match(i->second->name, parameters[0]) && !match(i->second->topic, parameters[0]))
+                       if (!InspIRCd::Match(i->second->name, parameters[0]) && !InspIRCd::Match(i->second->topic, parameters[0]))
                                continue;
                }
 
                // if the channel is not private/secret, OR the user is on the channel anyway
-               bool n = i->second->HasUser(user);
-               if ((i->second->IsModeSet('p')) && (!n))
+               bool n = (i->second->HasUser(user) || user->HasPrivPermission("channels/auspex"));
+
+               if (!n && i->second->IsModeSet('p'))
                {
-                       user->WriteServ("322 %s *",user->nick,i->second->name);
+                       /* Channel is +p and user is outside/not privileged */
+                       user->WriteNumeric(322, "%s * %ld :",user->nick.c_str(), users);
                }
                else
                {
-                       if (((!(i->second->IsModeSet('p'))) && (!(i->second->IsModeSet('s')))) || (n))
+                       if (n || !i->second->IsModeSet('s'))
                        {
-                               user->WriteServ("322 %s %s %d :[+%s] %s",user->nick,i->second->name,users,i->second->ChanModes(n),i->second->topic);
+                               /* User is in the channel/privileged, channel is not +s */
+                               user->WriteNumeric(322, "%s %s %ld :[+%s] %s",user->nick.c_str(),i->second->name.c_str(),users,i->second->ChanModes(n),i->second->topic.c_str());
                        }
                }
        }
-       user->WriteServ("323 %s :End of channel list.",user->nick);
+       user->WriteNumeric(323, "%s :End of channel list.",user->nick.c_str());
 
        return CMD_SUCCESS;
 }
 
+
+COMMAND_INIT(CommandList)