Log unknown class in auth errors to ircd.log as well.

[irc/rqf/shadowircd.git] / modules / m_scan.c

diff --git a/modules/m_scan.c b/modules/m_scan.c
index 4419fad5a08d0a302eec3189d620f5f9048145e2..7f4718761fc44215470e101c250c4d2246a61cd9 100644 (file)
--- a/modules/m_scan.c
+++ b/modules/m_scan.c
@@ -38,10 +38,9 @@
 #include "hash.h"
 #include "common.h"
 #include "hash.h"
-#include "irc_string.h"
+#include "match.h"
 #include "ircd.h"
 #include "numeric.h"
-#include "commio.h"
 #include "s_serv.h"
 #include "s_conf.h"
 #include "s_newconf.h"
@@ -80,7 +79,6 @@ static const char *spoofed_sockhost = "0";
 
 /*
  * m_scan
- *      parv[0] = sender prefix
  *      parv[1] = options [or target]
  *     parv[2] = [target]
  */
@@ -136,6 +134,12 @@ scan_umodes(struct Client *client_p, struct Client *source_p, int parc,
                return -1;
        }
 
+       if (parv[2][0] != '+' && parv[2][0] != '-')
+       {
+               sendto_one_notice(source_p, ":SCAN UMODES: umodes parameter must start with '+' or '-'");
+               return -1;
+       }
+
        for (c = parv[2]; *c; c++)
        {
                switch(*c)
@@ -171,19 +175,29 @@ scan_umodes(struct Client *client_p, struct Client *source_p, int parc,
                                list_max = atoi(parv[++i]);
                        else if (!irccmp(parv[i], "mask"))
                                mask = parv[++i];
+                       else
+                       {
+                               sendto_one_notice(source_p, ":SCAN UMODES: invalid parameters");
+                               return -1;
+                       }
+               }
+               else
+               {
+                       sendto_one_notice(source_p, ":SCAN UMODES: invalid parameters");
+                       return -1;
                }
        }
-       if (target_list == &global_client_list && (list_users || mask))
+       if (target_list == &global_client_list && list_users)
        {
                if (IsOperSpy(source_p))
                {
                        if (!ConfigFileEntry.operspy_dont_care_user_info)
                        {
-                               strlcpy(buf, "UMODES", sizeof buf);
+                               rb_strlcpy(buf, "UMODES", sizeof buf);
                                for (i = 2; i < parc; i++)
                                {
-                                       strlcat(buf, " ", sizeof buf);
-                                       strlcat(buf, parv[i], sizeof buf);
+                                       rb_strlcat(buf, " ", sizeof buf);
+                                       rb_strlcat(buf, parv[i], sizeof buf);
                                }
                                report_operspy(source_p, "SCAN", buf);
                        }
@@ -225,7 +239,7 @@ scan_umodes(struct Client *client_p, struct Client *source_p, int parc,
 
                if (mask != NULL)
                {
-                       ircsnprintf(maskbuf, BUFSIZE, "%s!%s@%s",
+                       rb_snprintf(maskbuf, BUFSIZE, "%s!%s@%s",
                                target_p->name, target_p->username, target_p->host);
 
                        if (!match(mask, maskbuf))