]> jfr.im git - solanum.git/blobdiff - modules/m_stats.c
Cleanup warnings
[solanum.git] / modules / m_stats.c
index 10b2ac4df41f24754080dd6491cff2f3e4f52065..b963d2ba44db6ce68a48c9c630ac4e0b0a696b44 100644 (file)
 static const char stats_desc[] =
        "Provides the STATS command to inspect various server/network information";
 
-static int m_stats (struct MsgBuf *, struct Client *, struct Client *, int, const char **);
+static void m_stats (struct MsgBuf *, struct Client *, struct Client *, int, const char **);
 
 struct Message stats_msgtab = {
-       "STATS", 0, 0, 0, 0,
+       "STATS", false, false, false, false,
        {mg_unreg, {m_stats, 2}, {m_stats, 3}, mg_ignore, mg_ignore, {m_stats, 2}}
 };
 
@@ -73,20 +73,27 @@ DECLARE_MODULE_AV2(stats, NULL, NULL, stats_clist, stats_hlist, NULL, NULL, NULL
 
 const char *Lformat = "%s %u %u %u %u %u :%u %u %s";
 
-static void stats_l_list(struct Client *s, const char *, int, int, rb_dlink_list *, char,
-                               int (*check_fn)(struct Client *target_p));
+static void stats_l_list(struct Client *s, const char *, bool, bool, rb_dlink_list *, char,
+                               bool (*check_fn)(struct Client *target_p));
 static void stats_l_client(struct Client *source_p, struct Client *target_p,
                                char statchar);
 
 static int stats_spy(struct Client *, char, const char *);
 static void stats_p_spy(struct Client *);
 
-/* Heres our struct for the stats table */
+typedef void (*handler_t)(struct Client *source_p);
+typedef void (*handler_parv_t)(struct Client *source_p, int parc, const char *parv[]);
+
 struct stats_cmd
 {
-       void (*handler) (struct Client *source_p);
-       int need_oper;
-       int need_admin;
+       union
+       {
+               handler_t handler;
+               handler_parv_t handler_parv;
+       };
+       bool need_parv;
+       bool need_oper;
+       bool need_admin;
 };
 
 static void stats_dns_servers(struct Client *);
@@ -130,58 +137,58 @@ static void stats_capability(struct Client *);
  * stats letter,  function to call, operonly? adminonly? --fl_
  *
  * Previously in this table letters were a column. I fixed it to use modern
- * C initalisers so we don't have to iterate anymore.
+ * C initalisers so we don't have to iterate anymore
  * --Elizafox
  */
-static struct stats_cmd stats_cmd_table[255] = {
-    /* letter     function     need_oper need_admin */
-       ['a'] = { stats_dns_servers,    1, 1, },
-       ['A'] = { stats_dns_servers,    1, 1, },
-       ['b'] = { stats_delay,          1, 1, },
-       ['B'] = { stats_hash,           1, 1, },
-       ['c'] = { stats_connect,        0, 0, },
-       ['C'] = { stats_capability,     1, 0, },
-       ['d'] = { stats_tdeny,          1, 0, },
-       ['D'] = { stats_deny,           1, 0, },
-       ['e'] = { stats_exempt,         1, 0, },
-       ['E'] = { stats_events,         1, 1, },
-       ['f'] = { stats_comm,           1, 1, },
-       ['F'] = { stats_comm,           1, 1, },
-       ['g'] = { stats_prop_klines,    1, 0, },
-       ['h'] = { stats_hubleaf,        0, 0, },
-       ['H'] = { stats_hubleaf,        0, 0, },
-       ['i'] = { stats_auth,           0, 0, },
-       ['I'] = { stats_auth,           0, 0, },
-       ['k'] = { stats_tklines,        0, 0, },
-       ['K'] = { stats_klines,         0, 0, },
-       ['l'] = { NULL /* special */,   0, 0, },
-       ['L'] = { NULL /* special */,   0, 0, },
-       ['m'] = { stats_messages,       0, 0, },
-       ['M'] = { stats_messages,       0, 0, },
-       ['n'] = { stats_dnsbl,          0, 0, },
-       ['o'] = { stats_oper,           0, 0, },
-       ['O'] = { stats_privset,        1, 0, },
-       ['p'] = { stats_operedup,       0, 0, },
-       ['P'] = { stats_ports,          0, 0, },
-       ['q'] = { stats_tresv,          1, 0, },
-       ['Q'] = { stats_resv,           1, 0, },
-       ['r'] = { stats_usage,          1, 0, },
-       ['R'] = { stats_usage,          1, 0, },
-       ['s'] = { stats_ssld,           1, 1, },
-       ['S'] = { stats_ssld,           1, 1, },
-       ['t'] = { stats_tstats,         1, 0, },
-       ['T'] = { stats_tstats,         1, 0, },
-       ['u'] = { stats_uptime,         0, 0, },
-       ['U'] = { stats_shared,         1, 0, },
-       ['v'] = { stats_servers,        0, 0, },
-       ['V'] = { stats_servers,        0, 0, },
-       ['x'] = { stats_tgecos,         1, 0, },
-       ['X'] = { stats_gecos,          1, 0, },
-       ['y'] = { stats_class,          0, 0, },
-       ['Y'] = { stats_class,          0, 0, },
-       ['z'] = { stats_memory,         1, 0, },
-       ['Z'] = { stats_ziplinks,       1, 0, },
-       ['?'] = { stats_servlinks,      0, 0, },
+static struct stats_cmd stats_cmd_table[256] = {
+/*     letter  handler/handler_parv                    parv    oper    admin   */
+       ['a'] = { { stats_dns_servers },                        false,  true,   true,   },
+       ['A'] = { { stats_dns_servers },                        false,  true,   true,   },
+       ['b'] = { { stats_delay },                              false,  true,   true,   },
+       ['B'] = { { stats_hash },                               false,  true,   true,   },
+       ['c'] = { { stats_connect },                    false,  false,  false,  },
+       ['C'] = { { stats_capability },                 false,  true,   false,  },
+       ['d'] = { { stats_tdeny },                              false,  true,   false,  },
+       ['D'] = { { stats_deny },                               false,  true,   false,  },
+       ['e'] = { { stats_exempt },                             false,  true,   false,  },
+       ['E'] = { { stats_events },                             false,  true,   true,   },
+       ['f'] = { { stats_comm },                               false,  true,   true,   },
+       ['F'] = { { stats_comm },                               false,  true,   true,   },
+       ['g'] = { { stats_prop_klines },                        false,  true,   false,  },
+       ['h'] = { { stats_hubleaf },                    false,  false,  false,  },
+       ['H'] = { { stats_hubleaf },                    false,  false,  false,  },
+       ['i'] = { { stats_auth },                               false,  false,  false,  },
+       ['I'] = { { stats_auth },                               false,  false,  false,  },
+       ['k'] = { { stats_tklines },                    false,  false,  false,  },
+       ['K'] = { { stats_klines },                             false,  false,  false,  },
+       ['l'] = { { .handler_parv = stats_ltrace },             true,   false,  false,  },
+       ['L'] = { { .handler_parv = stats_ltrace },             true,   false,  false,  },
+       ['m'] = { { stats_messages },                   false,  false,  false,  },
+       ['M'] = { { stats_messages },                   false,  false,  false,  },
+       ['n'] = { { stats_dnsbl },                              false,  false,  false,  },
+       ['o'] = { { stats_oper },                               false,  false,  false,  },
+       ['O'] = { { stats_privset },                    false,  true,   false,  },
+       ['p'] = { { stats_operedup },                   false,  false,  false,  },
+       ['P'] = { { stats_ports },                              false,  false,  false,  },
+       ['q'] = { { stats_tresv },                              false,  true,   false,  },
+       ['Q'] = { { stats_resv },                               false,  true,   false,  },
+       ['r'] = { { stats_usage },                              false,  true,   false,  },
+       ['R'] = { { stats_usage },                              false,  true,   false,  },
+       ['s'] = { { stats_ssld },                               false,  true,   true,   },
+       ['S'] = { { stats_ssld },                               false,  true,   true,   },
+       ['t'] = { { stats_tstats },                             false,  true,   false,  },
+       ['T'] = { { stats_tstats },                             false,  true,   false,  },
+       ['u'] = { { stats_uptime },                             false,  false,  false,  },
+       ['U'] = { { stats_shared },                             false,  true,   false,  },
+       ['v'] = { { stats_servers },                    false,  false,  false,  },
+       ['V'] = { { stats_servers },                    false,  false,  false,  },
+       ['x'] = { { stats_tgecos },                             false,  true,   false,  },
+       ['X'] = { { stats_gecos },                              false,  true,   false,  },
+       ['y'] = { { stats_class },                              false,  false,  false,  },
+       ['Y'] = { { stats_class },                              false,  false,  false,  },
+       ['z'] = { { stats_memory },                             false,  true,   false,  },
+       ['Z'] = { { stats_ziplinks },                   false,  true,   false,  },
+       ['?'] = { { stats_servlinks },                  false,  false,  false,  },
 };
 
 /*
@@ -193,13 +200,12 @@ static struct stats_cmd stats_cmd_table[255] = {
  * This will search the tables for the appropriate stats letter,
  * if found execute it.
  */
-static int
+static void
 m_stats(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_p, int parc, const char *parv[])
 {
        static time_t last_used = 0;
-       int i;
        struct stats_cmd *cmd;
-       char statchar;
+       unsigned char statchar;
        int did_stats = 0;
 
        statchar = parv[1][0];
@@ -214,25 +220,18 @@ m_stats(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_
                                   me.name, source_p->name, "STATS");
                        sendto_one_numeric(source_p, RPL_ENDOFSTATS,
                                           form_str(RPL_ENDOFSTATS), statchar);
-                       return 0;
+                       return;
                }
                else
                        last_used = rb_current_time();
        }
 
        if(hunt_server (client_p, source_p, ":%s STATS %s :%s", 2, parc, parv) != HUNTED_ISME)
-               return 0;
+               return;
 
-       if((statchar != 'L') && (statchar != 'l'))
-       {
+       if(tolower(statchar) != 'l')
+               /* FIXME */
                did_stats = stats_spy(source_p, statchar, NULL);
-       }
-       else
-       {
-               /* Blah, stats L needs the parameters, none of the others do.. */
-               stats_ltrace (source_p, parc, parv);
-               goto stats_out;
-       }
 
        /* if did_stats is true, a module grabbed this STATS request */
        if(did_stats)
@@ -259,15 +258,16 @@ m_stats(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_
                        goto stats_out;
                }
 
-               cmd->handler(source_p);
+               if(cmd->need_parv)
+                       cmd->handler_parv(source_p, parc, parv);
+               else
+                       cmd->handler(source_p);
        }
 
 stats_out:
        /* Send the end of stats notice, and the stats_spy */
        sendto_one_numeric(source_p, RPL_ENDOFSTATS,
                           form_str(RPL_ENDOFSTATS), statchar);
-
-       return 0;
 }
 
 static void
@@ -565,7 +565,7 @@ stats_auth (struct Client *source_p)
                        aconf = find_conf_by_address (source_p->host, source_p->sockhost, NULL,
                                                      (struct sockaddr *)&source_p->localClient->ip,
                                                      CONF_CLIENT,
-                                                     source_p->localClient->ip.ss_family,
+                                                     GET_SS_FAMILY(&source_p->localClient->ip),
                                                      source_p->username, NULL);
                else
                        aconf = find_conf_by_address (source_p->host, NULL, NULL, NULL, CONF_CLIENT,
@@ -607,7 +607,7 @@ stats_tklines(struct Client *source_p)
                        aconf = find_conf_by_address (source_p->host, source_p->sockhost, NULL,
                                                      (struct sockaddr *)&source_p->localClient->ip,
                                                      CONF_KILL,
-                                                     source_p->localClient->ip.ss_family,
+                                                     GET_SS_FAMILY(&source_p->localClient->ip),
                                                      source_p->username, NULL);
                else
                        aconf = find_conf_by_address (source_p->host, NULL, NULL, NULL, CONF_KILL,
@@ -710,7 +710,7 @@ stats_klines(struct Client *source_p)
                        aconf = find_conf_by_address (source_p->host, source_p->sockhost, NULL,
                                                      (struct sockaddr *)&source_p->localClient->ip,
                                                      CONF_KILL,
-                                                     source_p->localClient->ip.ss_family,
+                                                     GET_SS_FAMILY(&source_p->localClient->ip),
                                                      source_p->username, NULL);
                else
                        aconf = find_conf_by_address (source_p->host, NULL, NULL, NULL, CONF_KILL,
@@ -872,7 +872,6 @@ stats_tresv(struct Client *source_p)
        struct ConfItem *aconf;
        struct rb_radixtree_iteration_state state;
        rb_dlink_node *ptr;
-       int i;
 
        RB_DLINK_FOREACH(ptr, resv_conf_list.head)
        {
@@ -899,7 +898,6 @@ stats_resv(struct Client *source_p)
        struct ConfItem *aconf;
        struct rb_radixtree_iteration_state state;
        rb_dlink_node *ptr;
-       int i;
 
        RB_DLINK_FOREACH(ptr, resv_conf_list.head)
        {
@@ -941,6 +939,7 @@ stats_ssld(struct Client *source_p)
 static void
 stats_usage (struct Client *source_p)
 {
+#ifndef _WIN32
        struct rusage rus;
        time_t secs;
        time_t rup;
@@ -992,6 +991,7 @@ stats_usage (struct Client *source_p)
                           "R :Signals %d Context Vol. %d Invol %d",
                           (int) rus.ru_nsignals, (int) rus.ru_nvcsw,
                           (int) rus.ru_nivcsw);
+#endif
 }
 
 static void
@@ -1587,20 +1587,17 @@ stats_servlinks (struct Client *source_p)
        sendto_one_numeric(source_p, RPL_STATSDEBUG, "? :Server recv: %s", buf);
 }
 
-static int
+static inline bool
 stats_l_should_show_oper(struct Client *target_p)
 {
-       if (IsOperInvis(target_p))
-               return 0;
-
-       return 1;
+       return (!IsOperInvis(target_p));
 }
 
 static void
 stats_ltrace(struct Client *source_p, int parc, const char *parv[])
 {
-       int doall = 0;
-       int wilds = 0;
+       bool doall = false;
+       bool wilds = false;
        const char *name;
        char statchar = parv[1][0];
 
@@ -1612,7 +1609,7 @@ stats_ltrace(struct Client *source_p, int parc, const char *parv[])
                   (!MyClient(source_p) && !irccmp(parv[2], me.id)))
                {
                        name = me.name;
-                       doall = 1;
+                       doall = true;
                }
                else
                {
@@ -1646,7 +1643,7 @@ stats_ltrace(struct Client *source_p, int parc, const char *parv[])
        else
        {
                name = me.name;
-               doall = 1;
+               doall = true;
        }
 
        stats_spy(source_p, statchar, name);
@@ -1684,8 +1681,8 @@ stats_ltrace(struct Client *source_p, int parc, const char *parv[])
 }
 
 static void
-stats_l_list(struct Client *source_p, const char *name, int doall, int wilds,
-            rb_dlink_list * list, char statchar, int (*check_fn)(struct Client *target_p))
+stats_l_list(struct Client *source_p, const char *name, bool doall, bool wilds,
+            rb_dlink_list * list, char statchar, bool (*check_fn)(struct Client *target_p))
 {
        rb_dlink_node *ptr;
        struct Client *target_p;
@@ -1766,10 +1763,6 @@ stats_comm(struct Client *source_p)
  * output      - none
  * side effects -
  * This little helper function reports to opers if configured.
- * personally, I don't see why opers need to see stats requests
- * at all. They are just "noise" to an oper, and users can't do
- * any damage with stats requests now anyway. So, why show them?
- * -Dianora
  */
 static int
 stats_spy(struct Client *source_p, char statchar, const char *name)