]> jfr.im git - solanum.git/blobdiff - modules/m_xline.c
Fix m_xline [ci skip]
[solanum.git] / modules / m_xline.c
index 50423715e2de8e00873bec7ed9bc4a19307dd21f..02c162af44019c79baaa1a51e38c1b07b5f8dd07 100644 (file)
 #include "bandbi.h"
 #include "operhash.h"
 
-static int mo_xline(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_p, int parc, const char *parv[]);
-static int ms_xline(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_p, int parc, const char *parv[]);
-static int me_xline(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_p, int parc, const char *parv[]);
-static int mo_unxline(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_p, int parc,
+static const char xline_desc[] =
+       "Provides management of GECOS bans via (UN)XLINE command";
+
+static void mo_xline(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_p, int parc, const char *parv[]);
+static void ms_xline(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_p, int parc, const char *parv[]);
+static void me_xline(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_p, int parc, const char *parv[]);
+static void mo_unxline(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_p, int parc,
                      const char *parv[]);
-static int ms_unxline(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_p, int parc,
+static void ms_unxline(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_p, int parc,
                      const char *parv[]);
-static int me_unxline(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_p, int parc,
+static void me_unxline(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_p, int parc,
                      const char *parv[]);
 
+static bool valid_xline(struct Client *, const char *, const char *);
+static void apply_xline(struct Client *client_p, const char *name,
+                       const char *reason, int temp_time, bool propagated);
+static void propagate_xline(struct Client *source_p, const char *target,
+                           int temp_time, const char *name, const char *type, const char *reason);
+static void cluster_xline(struct Client *source_p, int temp_time,
+                         const char *name, const char *reason);
+
+static void handle_remote_xline(struct Client *source_p, int temp_time,
+                               const char *name, const char *reason);
+static void handle_remote_unxline(struct Client *source_p, const char *name);
+static void remove_xline(struct Client *source_p, const char *name,
+                        bool propagated);
+
 struct Message xline_msgtab = {
        "XLINE", 0, 0, 0, 0,
        {mg_unreg, mg_not_oper, {ms_xline, 5}, {ms_xline, 5}, {me_xline, 5}, {mo_xline, 3}}
@@ -73,23 +90,7 @@ struct Message unxline_msgtab = {
 
 mapi_clist_av1 xline_clist[] = { &xline_msgtab, &unxline_msgtab, NULL };
 
-DECLARE_MODULE_AV1(xline, NULL, NULL, xline_clist, NULL, NULL, "$Revision$");
-
-static int valid_xline(struct Client *, const char *, const char *);
-static void apply_xline(struct Client *client_p, const char *name,
-                       const char *reason, int temp_time, int propagated);
-static void propagate_xline(struct Client *source_p, const char *target,
-                           int temp_time, const char *name, const char *type, const char *reason);
-static void cluster_xline(struct Client *source_p, int temp_time,
-                         const char *name, const char *reason);
-
-static void handle_remote_xline(struct Client *source_p, int temp_time,
-                               const char *name, const char *reason);
-static void handle_remote_unxline(struct Client *source_p, const char *name);
-
-static void remove_xline(struct Client *source_p, const char *name,
-                        int propagated);
-
+DECLARE_MODULE_AV2(xline, NULL, NULL, xline_clist, NULL, NULL, NULL, NULL, xline_desc);
 
 /* m_xline()
  *
@@ -97,7 +98,7 @@ static void remove_xline(struct Client *source_p, const char *name,
  * parv[2] - optional type/reason
  * parv[3] - reason
  */
-static int
+static void
 mo_xline(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_p, int parc, const char *parv[])
 {
        struct ConfItem *aconf;
@@ -106,12 +107,12 @@ mo_xline(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source
        const char *target_server = NULL;
        int temp_time;
        int loc = 1;
-       int propagated = ConfigFileEntry.use_propagated_bans;
+       bool propagated = ConfigFileEntry.use_propagated_bans;
 
        if(!IsOperXline(source_p))
        {
                sendto_one(source_p, form_str(ERR_NOPRIVS), me.name, source_p->name, "xline");
-               return 0;
+               return;
        }
 
        if((temp_time = valid_temp_time(parv[loc])) >= 0)
@@ -130,7 +131,7 @@ mo_xline(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source
                {
                        sendto_one(source_p, form_str(ERR_NOPRIVS),
                                   me.name, source_p->name, "remoteban");
-                       return 0;
+                       return;
                }
 
                target_server = parv[loc + 1];
@@ -141,7 +142,7 @@ mo_xline(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source
        {
                sendto_one(source_p, form_str(ERR_NEEDMOREPARAMS),
                           me.name, source_p->name, "XLINE");
-               return 0;
+               return;
        }
 
        reason = parv[loc];
@@ -151,10 +152,10 @@ mo_xline(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source
                propagate_xline(source_p, target_server, temp_time, name, "2", reason);
 
                if(!match(target_server, me.name))
-                       return 0;
+                       return;
 
                /* Set as local-only. */
-               propagated = 0;
+               propagated = false;
        }
        else if(!propagated && rb_dlink_list_length(&cluster_conf_list) > 0)
                cluster_xline(source_p, temp_time, name, reason);
@@ -163,28 +164,26 @@ mo_xline(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source
        {
                sendto_one(source_p, ":%s NOTICE %s :[%s] already X-Lined by [%s] - %s",
                           me.name, source_p->name, name, aconf->host, aconf->passwd);
-               return 0;
+               return;
        }
 
        if(!valid_xline(source_p, name, reason))
-               return 0;
+               return;
 
        if(propagated && temp_time == 0)
        {
                sendto_one_notice(source_p, ":Cannot set a permanent global ban");
-               return 0;
+               return;
        }
 
        apply_xline(source_p, name, reason, temp_time, propagated);
-
-       return 0;
 }
 
 /* ms_xline()
  *
  * handles a remote xline
  */
-static int
+static void
 ms_xline(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_p, int parc, const char *parv[])
 {
        /* parv[0]  parv[1]      parv[2]  parv[3]  parv[4]
@@ -193,25 +192,23 @@ ms_xline(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source
        propagate_xline(source_p, parv[1], 0, parv[2], parv[3], parv[4]);
 
        if(!IsPerson(source_p))
-               return 0;
+               return;
 
        /* destined for me? */
        if(!match(parv[1], me.name))
-               return 0;
+               return;
 
        handle_remote_xline(source_p, 0, parv[2], parv[4]);
-       return 0;
 }
 
-static int
+static void
 me_xline(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_p, int parc, const char *parv[])
 {
        /* time name type :reason */
        if(!IsPerson(source_p))
-               return 0;
+               return;
 
        handle_remote_xline(source_p, atoi(parv[1]), parv[2], parv[4]);
-       return 0;
 }
 
 static void
@@ -235,7 +232,7 @@ handle_remote_xline(struct Client *source_p, int temp_time, const char *name, co
                return;
        }
 
-       apply_xline(source_p, name, reason, temp_time, 0);
+       apply_xline(source_p, name, reason, temp_time, false);
 }
 
 /* valid_xline()
@@ -244,14 +241,14 @@ handle_remote_xline(struct Client *source_p, int temp_time, const char *name, co
  * outputs     -
  * side effects - checks the xline for validity, erroring if needed
  */
-static int
+static bool
 valid_xline(struct Client *source_p, const char *gecos, const char *reason)
 {
        if(EmptyString(reason))
        {
                sendto_one(source_p, form_str(ERR_NEEDMOREPARAMS),
                           get_id(&me, source_p), get_id(source_p, source_p), "XLINE");
-               return 0;
+               return false;
        }
 
        if(!valid_wild_card_simple(gecos))
@@ -260,14 +257,14 @@ valid_xline(struct Client *source_p, const char *gecos, const char *reason)
                                  ":Please include at least %d non-wildcard "
                                  "characters with the xline",
                                  ConfigFileEntry.min_nonwildcard_simple);
-               return 0;
+               return false;
        }
 
-       return 1;
+       return true;
 }
 
 void
-apply_xline(struct Client *source_p, const char *name, const char *reason, int temp_time, int propagated)
+apply_xline(struct Client *source_p, const char *name, const char *reason, int temp_time, bool propagated)
 {
        struct ConfItem *aconf;
 
@@ -385,15 +382,15 @@ cluster_xline(struct Client *source_p, int temp_time, const char *name, const ch
  *
  * parv[1] - thing to unxline
  */
-static int
+static void
 mo_unxline(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_p, int parc, const char *parv[])
 {
-       int propagated = 1;
+       bool propagated = true;
 
        if(!IsOperXline(source_p))
        {
                sendto_one(source_p, form_str(ERR_NOPRIVS), me.name, source_p->name, "xline");
-               return 0;
+               return;
        }
 
        if(parc == 4 && !(irccmp(parv[2], "ON")))
@@ -402,28 +399,26 @@ mo_unxline(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *sour
                {
                        sendto_one(source_p, form_str(ERR_NOPRIVS),
                                   me.name, source_p->name, "remoteban");
-                       return 0;
+                       return;
                }
 
                propagate_generic(source_p, "UNXLINE", parv[3], CAP_CLUSTER, "%s", parv[1]);
 
                if(match(parv[3], me.name) == 0)
-                       return 0;
+                       return;
 
-               propagated = 0;
+               propagated = false;
        }
        /* cluster{} moved to remove_xline */
 
        remove_xline(source_p, parv[1], propagated);
-
-       return 0;
 }
 
 /* ms_unxline()
  *
  * handles a remote unxline
  */
-static int
+static void
 ms_unxline(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_p, int parc, const char *parv[])
 {
        /* parv[0]  parv[1]        parv[2]
@@ -432,24 +427,22 @@ ms_unxline(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *sour
        propagate_generic(source_p, "UNXLINE", parv[1], CAP_CLUSTER, "%s", parv[2]);
 
        if(!match(parv[1], me.name))
-               return 0;
+               return;
 
        if(!IsPerson(source_p))
-               return 0;
+               return;
 
        handle_remote_unxline(source_p, parv[2]);
-       return 0;
 }
 
-static int
+static void
 me_unxline(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_p, int parc, const char *parv[])
 {
        /* name */
        if(!IsPerson(source_p))
-               return 0;
+               return;
 
        handle_remote_unxline(source_p, parv[1]);
-       return 0;
 }
 
 static void
@@ -459,13 +452,11 @@ handle_remote_unxline(struct Client *source_p, const char *name)
                             source_p->servptr->name, SHARED_UNXLINE))
                return;
 
-       remove_xline(source_p, name, 0);
-
-       return;
+       remove_xline(source_p, name, false);
 }
 
 static void
-remove_xline(struct Client *source_p, const char *name, int propagated)
+remove_xline(struct Client *source_p, const char *name, bool propagated)
 {
        struct ConfItem *aconf;
        rb_dlink_node *ptr;
@@ -543,6 +534,4 @@ remove_xline(struct Client *source_p, const char *name, int propagated)
                cluster_generic(source_p, "UNXLINE", SHARED_UNXLINE, CAP_CLUSTER, "%s", name);
 
        sendto_one_notice(source_p, ":No X-Line for %s", name);
-
-       return;
 }