]> jfr.im git - solanum.git/blobdiff - modules/core/m_join.c
Don't send original error message if we're already on the channel we're forwarding to
[solanum.git] / modules / core / m_join.c
index bba9eaf141ced267857c408ad721fb5be08f2a3b..5ed6cb29361cf658aeaca828e0b8df8045c2ce41 100644 (file)
  *  along with this program; if not, write to the Free Software
  *  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307
  *  USA
- *
- *  $Id: m_join.c 3494 2007-05-27 13:07:27Z jilles $
  */
 
 #include "stdinc.h"
 #include "channel.h"
 #include "client.h"
-#include "common.h"
 #include "hash.h"
 #include "match.h"
 #include "ircd.h"
 #include "packet.h"
 #include "chmode.h"
 #include "ratelimit.h"
+#include "s_assert.h"
+
+static const char join_desc[] = "Provides the JOIN and TS6 SJOIN commands to facilitate joining and creating channels";
 
-static int m_join(struct Client *, struct Client *, int, const char **);
-static int ms_join(struct Client *, struct Client *, int, const char **);
-static int ms_sjoin(struct Client *, struct Client *, int, const char **);
+static void m_join(struct MsgBuf *, struct Client *, struct Client *, int, const char **);
+static void ms_join(struct MsgBuf *, struct Client *, struct Client *, int, const char **);
+static void ms_sjoin(struct MsgBuf *, struct Client *, struct Client *, int, const char **);
 
 static int h_can_create_channel;
 static int h_channel_join;
 
 struct Message join_msgtab = {
-       "JOIN", 0, 0, 0, MFLG_SLOW,
+       "JOIN", 0, 0, 0, 0,
        {mg_unreg, {m_join, 2}, {ms_join, 2}, mg_ignore, mg_ignore, {m_join, 2}}
 };
 
 struct Message sjoin_msgtab = {
-       "SJOIN", 0, 0, 0, MFLG_SLOW,
+       "SJOIN", 0, 0, 0, 0,
        {mg_unreg, mg_ignore, mg_ignore, {ms_sjoin, 4}, mg_ignore, mg_ignore}
 };
 
@@ -68,10 +68,11 @@ mapi_hlist_av1 join_hlist[] = {
        { NULL, NULL },
 };
 
-DECLARE_MODULE_AV1(join, NULL, NULL, join_clist, join_hlist, NULL, "$Revision: 3494 $");
+DECLARE_MODULE_AV2(join, NULL, NULL, join_clist, join_hlist, NULL, NULL, NULL, join_desc);
 
 static void do_join_0(struct Client *client_p, struct Client *source_p);
-static int check_channel_name_loc(struct Client *source_p, const char *name);
+static bool check_channel_name_loc(struct Client *source_p, const char *name);
+static void send_join_error(struct Client *source_p, int numeric, const char *name);
 
 static void set_final_mode(struct Mode *mode, struct Mode *oldmode);
 static void remove_our_modes(struct Channel *chptr, struct Client *source_p);
@@ -109,19 +110,28 @@ check_forward(struct Client *source_p, struct Channel *chptr,
        {
                if (next == NULL)
                        return NULL;
+
                chptr = find_channel(next);
                /* Can only forward to existing channels */
                if (chptr == NULL)
                        return NULL;
-               /* Already on there, show original error message */
+               /* Already on there... but don't send the original reason for
+                * being unable to join. It isn't their fault they're already
+                * on the channel, and it looks hostile otherwise.
+                * --Elizafox
+                */
                if (IsMember(source_p, chptr))
+               {
+                       *err = ERR_USERONCHANNEL; /* I'm borrowing this for now. --Elizafox */
                        return NULL;
+               }
                /* Juped. Sending a warning notice would be unfair */
                if (hash_find_resv(chptr->chname))
                        return NULL;
                /* Don't forward to +Q channel */
                if (chptr->mode.mode & MODE_DISFORWARD)
                        return NULL;
+
                i = can_join(source_p, chptr, key, &next);
                if (i == 0)
                        return chptr;
@@ -136,8 +146,8 @@ check_forward(struct Client *source_p, struct Channel *chptr,
  *      parv[1] = channel
  *      parv[2] = channel password (key)
  */
-static int
-m_join(struct Client *client_p, struct Client *source_p, int parc, const char *parv[])
+static void
+m_join(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_p, int parc, const char *parv[])
 {
        static char jbuf[BUFSIZE];
        struct Channel *chptr = NULL, *chptr2 = NULL;
@@ -256,10 +266,10 @@ m_join(struct Client *client_p, struct Client *source_p, int parc, const char *p
 
                        if(moduledata.approved != 0)
                        {
-#ifdef XXX_NOTYET
-                               sendto_one(source_p, form_str(moduledata.approved),
-                                          me.name, source_p->name, name);
-#endif
+                               if(moduledata.approved != ERR_CUSTOM)
+                                       send_join_error(source_p,
+                                                       moduledata.approved,
+                                                       name);
                                continue;
                        }
 
@@ -276,9 +286,9 @@ m_join(struct Client *client_p, struct Client *source_p, int parc, const char *p
 
                if((rb_dlink_list_length(&source_p->user->channel) >=
                    (unsigned long) ConfigChannel.max_chans_per_user) &&
-                  (!IsOper(source_p) ||
+                  (!IsExtendChans(source_p) ||
                    (rb_dlink_list_length(&source_p->user->channel) >=
-                    (unsigned long) ConfigChannel.max_chans_per_user * 3)))
+                    (unsigned long) ConfigChannel.max_chans_per_user_large)))
                {
                        sendto_one(source_p, form_str(ERR_TOOMANYCHANNELS),
                                   me.name, source_p->name, name);
@@ -298,16 +308,14 @@ m_join(struct Client *client_p, struct Client *source_p, int parc, const char *p
                }
 
                /* If check_forward returns NULL, they couldn't join and there wasn't a usable forward channel. */
-               if(!(chptr2 = check_forward(source_p, chptr, key, &i)))
+               if((chptr2 = check_forward(source_p, chptr, key, &i)) == NULL)
                {
                        /* might be wrong, but is there any other better location for such?
                         * see extensions/chm_operonly.c for other comments on this
                         * -- dwr
                         */
-#ifdef XXX_NOTYET
                        if(i != ERR_CUSTOM)
-                               sendto_one(source_p, form_str(i), me.name, source_p->name, name);
-#endif
+                               send_join_error(source_p, i, name);
                        continue;
                }
                else if(chptr != chptr2)
@@ -341,8 +349,7 @@ m_join(struct Client *client_p, struct Client *source_p, int parc, const char *p
                if(flags & CHFL_CHANOP)
                {
                        chptr->channelts = rb_current_time();
-                       chptr->mode.mode |= MODE_TOPICLIMIT;
-                       chptr->mode.mode |= MODE_NOPRIVMSGS;
+                       chptr->mode.mode |= ConfigChannel.autochanmodes;
                        modes = channel_modes(chptr, &me);
 
                        sendto_channel_local(ONLY_CHANOPS, chptr, ":%s MODE %s %s",
@@ -381,8 +388,6 @@ m_join(struct Client *client_p, struct Client *source_p, int parc, const char *p
                hook_info.key = key;
                call_hook(h_channel_join, &hook_info);
        }
-
-       return 0;
 }
 
 /*
@@ -392,53 +397,44 @@ m_join(struct Client *client_p, struct Client *source_p, int parc, const char *p
  *      parv[3] = "+", formerly channel modes but now unused
  * alternatively, a single "0" parameter parts all channels
  */
-static int
-ms_join(struct Client *client_p, struct Client *source_p, int parc, const char *parv[])
+static void
+ms_join(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_p, int parc, const char *parv[])
 {
        struct Channel *chptr;
        static struct Mode mode;
        time_t oldts;
        time_t newts;
-       int isnew;
-       int keep_our_modes = YES;
+       bool isnew;
+       bool keep_our_modes = true;
        rb_dlink_node *ptr, *next_ptr;
 
        /* special case for join 0 */
        if((parv[1][0] == '0') && (parv[1][1] == '\0') && parc == 2)
        {
                do_join_0(client_p, source_p);
-               return 0;
+               return;
        }
 
        if(parc < 4)
-               return 0;
+               return;
 
        if(!IsChannelName(parv[2]) || !check_channel_name(parv[2]))
-               return 0;
+               return;
 
        /* joins for local channels cant happen. */
        if(parv[2][0] == '&')
-               return 0;
+               return;
 
        mbuf = modebuf;
        mode.key[0] = mode.forward[0] = '\0';
        mode.mode = mode.limit = mode.join_num = mode.join_time = 0;
 
        if((chptr = get_or_create_channel(source_p, parv[2], &isnew)) == NULL)
-               return 0;
+               return;
 
        newts = atol(parv[1]);
        oldts = chptr->channelts;
 
-#ifdef IGNORE_BOGUS_TS
-       if(newts < 800000000)
-       {
-               sendto_realops_snomask(SNO_DEBUG, L_ALL,
-                                    "*** Bogus TS %ld on %s ignored from %s",
-                                    (long) newts, chptr->chname, client_p->name);
-               newts = (oldts == 0) ? oldts : 800000000;
-       }
-#else
        /* making a channel TS0 */
        if(!isnew && !newts && oldts)
        {
@@ -449,7 +445,6 @@ ms_join(struct Client *client_p, struct Client *source_p, int parc, const char *
                                     "Server %s changing TS on %s from %ld to 0",
                                     source_p->name, chptr->chname, (long) oldts);
        }
-#endif
 
        if(isnew)
                chptr->channelts = newts;
@@ -459,7 +454,7 @@ ms_join(struct Client *client_p, struct Client *source_p, int parc, const char *
                ;
        else if(newts < oldts)
        {
-               keep_our_modes = NO;
+               keep_our_modes = false;
                chptr->channelts = newts;
        }
 
@@ -490,7 +485,7 @@ ms_join(struct Client *client_p, struct Client *source_p, int parc, const char *
                *modebuf = *parabuf = '\0';
 
                /* since we're dropping our modes, we want to clear the mlock as well. --nenolod */
-               set_channel_mlock(client_p, source_p, chptr, NULL, FALSE);
+               set_channel_mlock(client_p, source_p, chptr, NULL, false);
        }
 
        if(!IsMember(source_p, chptr))
@@ -509,11 +504,10 @@ ms_join(struct Client *client_p, struct Client *source_p, int parc, const char *
        sendto_server(client_p, chptr, CAP_TS6, NOCAPS,
                      ":%s JOIN %ld %s +",
                      source_p->id, (long) chptr->channelts, chptr->chname);
-       return 0;
 }
 
-static int
-ms_sjoin(struct Client *client_p, struct Client *source_p, int parc, const char *parv[])
+static void
+ms_sjoin(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_p, int parc, const char *parv[])
 {
        static char buf_uid[BUFSIZE];
        static const char empty_modes[] = "0";
@@ -524,12 +518,11 @@ ms_sjoin(struct Client *client_p, struct Client *source_p, int parc, const char
        static struct Mode mode, *oldmode;
        const char *modes;
        int args = 0;
-       int keep_our_modes = 1;
-       int keep_new_modes = 1;
+       bool keep_our_modes = true;
+       bool keep_new_modes = true;
        int fl;
-       int isnew;
+       bool isnew;
        int mlen_uid;
-       int len_nick;
        int len_uid;
        int len;
        int joins = 0;
@@ -540,12 +533,15 @@ ms_sjoin(struct Client *client_p, struct Client *source_p, int parc, const char
        static char empty[] = "";
        rb_dlink_node *ptr, *next_ptr;
 
+       if(parc < 5)
+               return;
+
        if(!IsChannelName(parv[2]) || !check_channel_name(parv[2]))
-               return 0;
+               return;
 
        /* SJOIN's for local channels can't happen. */
        if(*parv[2] == '&')
-               return 0;
+               return;
 
        modebuf[0] = parabuf[0] = mode.key[0] = mode.forward[0] = '\0';
        pargs = mode.mode = mode.limit = mode.join_num = mode.join_time = 0;
@@ -568,7 +564,7 @@ ms_sjoin(struct Client *client_p, struct Client *source_p, int parc, const char
                        rb_strlcpy(mode.forward, parv[4 + args], sizeof(mode.forward));
                        args++;
                        if(parc < 5 + args)
-                               return 0;
+                               return;
                        break;
                case 'j':
                        sscanf(parv[4 + args], "%d:%d", &joinc, &timeslice);
@@ -576,19 +572,19 @@ ms_sjoin(struct Client *client_p, struct Client *source_p, int parc, const char
                        mode.join_num = joinc;
                        mode.join_time = timeslice;
                        if(parc < 5 + args)
-                               return 0;
+                               return;
                        break;
                case 'k':
                        rb_strlcpy(mode.key, parv[4 + args], sizeof(mode.key));
                        args++;
                        if(parc < 5 + args)
-                               return 0;
+                               return;
                        break;
                case 'l':
                        mode.limit = atoi(parv[4 + args]);
                        args++;
                        if(parc < 5 + args)
-                               return 0;
+                               return;
                        break;
                default:
                        if(chmode_flags[(int) *s] != 0)
@@ -610,22 +606,12 @@ ms_sjoin(struct Client *client_p, struct Client *source_p, int parc, const char
                s = "";
 
        if((chptr = get_or_create_channel(source_p, parv[2], &isnew)) == NULL)
-               return 0;       /* channel name too long? */
+               return; /* channel name too long? */
 
 
        oldts = chptr->channelts;
        oldmode = &chptr->mode;
 
-#ifdef IGNORE_BOGUS_TS
-       if(newts < 800000000)
-       {
-               sendto_realops_snomask(SNO_DEBUG, L_ALL,
-                                    "*** Bogus TS %ld on %s ignored from %s",
-                                    (long) newts, chptr->chname, client_p->name);
-
-               newts = (oldts == 0) ? oldts : 800000000;
-       }
-#else
        if(!isnew && !newts && oldts)
        {
                sendto_channel_local(ALL_MEMBERS, chptr,
@@ -636,11 +622,10 @@ ms_sjoin(struct Client *client_p, struct Client *source_p, int parc, const char
                                     "Server %s changing TS on %s from %ld to 0",
                                     source_p->name, chptr->chname, (long) oldts);
        }
-#endif
 
        if(isnew)
                chptr->channelts = newts;
-       
+
        else if(newts == 0 || oldts == 0)
                chptr->channelts = 0;
        else if(newts == oldts)
@@ -683,16 +668,16 @@ ms_sjoin(struct Client *client_p, struct Client *source_p, int parc, const char
                        {
                                /* Channel was emptied, create a new one */
                                if((chptr = get_or_create_channel(source_p, parv[2], &isnew)) == NULL)
-                                       return 0;               /* oops! */
+                                       return;         /* oops! */
 
                                oldmode = &chptr->mode;
                        }
                }
-               keep_our_modes = NO;
+               keep_our_modes = false;
                chptr->channelts = newts;
        }
        else
-               keep_new_modes = NO;
+               keep_new_modes = false;
 
        if(!keep_new_modes)
                mode = *oldmode;
@@ -754,7 +739,7 @@ ms_sjoin(struct Client *client_p, struct Client *source_p, int parc, const char
                strcpy(chptr->chname, parv[2]);
 
                /* since we're dropping our modes, we want to clear the mlock as well. --nenolod */
-               set_channel_mlock(client_p, source_p, chptr, NULL, FALSE);
+               set_channel_mlock(client_p, source_p, chptr, NULL, false);
        }
 
        if(*modebuf != '\0')
@@ -768,14 +753,14 @@ ms_sjoin(struct Client *client_p, struct Client *source_p, int parc, const char
        else
                modes = empty_modes;
 
-       mlen_uid = rb_sprintf(buf_uid, ":%s SJOIN %ld %s %s :",
+       mlen_uid = sprintf(buf_uid, ":%s SJOIN %ld %s %s :",
                              use_id(source_p), (long) chptr->channelts, parv[2], modes);
        ptr_uid = buf_uid + mlen_uid;
 
        mbuf = modebuf;
        para[0] = para[1] = para[2] = para[3] = empty;
        pargs = 0;
-       len_nick = len_uid = 0;
+       len_uid = 0;
 
        /* if theres a space, theres going to be more than one nick, change the
         * first space to \0, so s is just the first nick, and point p to the
@@ -827,19 +812,17 @@ ms_sjoin(struct Client *client_p, struct Client *source_p, int parc, const char
                        if(fl & CHFL_CHANOP)
                        {
                                *ptr_uid++ = '@';
-                               len_nick++;
                                len_uid++;
                        }
                        if(fl & CHFL_VOICE)
                        {
                                *ptr_uid++ = '+';
-                               len_nick++;
                                len_uid++;
                        }
                }
 
                /* copy the nick to the two buffers */
-               len = rb_sprintf(ptr_uid, "%s ", use_id(target_p));
+               len = sprintf(ptr_uid, "%s ", use_id(target_p));
                ptr_uid += len;
                len_uid += len;
 
@@ -937,7 +920,7 @@ ms_sjoin(struct Client *client_p, struct Client *source_p, int parc, const char
        {
                destroy_channel(chptr);
 
-               return 0;
+               return;
        }
 
        /* Keep the colon if we're sending an SJOIN without nicks -- jilles */
@@ -947,8 +930,6 @@ ms_sjoin(struct Client *client_p, struct Client *source_p, int parc, const char
        }
 
        sendto_server(client_p->from, NULL, CAP_TS6, NOCAPS, "%s", buf_uid);
-
-       return 0;
 }
 
 /*
@@ -987,21 +968,21 @@ do_join_0(struct Client *client_p, struct Client *source_p)
        }
 }
 
-static int
+static bool
 check_channel_name_loc(struct Client *source_p, const char *name)
 {
        const char *p;
 
        s_assert(name != NULL);
        if(EmptyString(name))
-               return 0;
+               return false;
 
        if(ConfigFileEntry.disable_fake_channels && !IsOper(source_p))
        {
                for(p = name; *p; ++p)
                {
                        if(!IsChanChar(*p) || IsFakeChanChar(*p))
-                               return 0;
+                               return false;
                }
        }
        else
@@ -1009,7 +990,7 @@ check_channel_name_loc(struct Client *source_p, const char *name)
                for(p = name; *p; ++p)
                {
                        if(!IsChanChar(*p))
-                               return 0;
+                               return false;
                }
        }
 
@@ -1017,10 +998,44 @@ check_channel_name_loc(struct Client *source_p, const char *name)
        {
                for(p = name; *p; ++p)
                        if(*p < 33 || *p > 126)
-                               return 0;
+                               return false;
        }
 
-       return 1;
+       return true;
+}
+
+/* send_join_error()
+ *
+ * input       - client to send to, reason, channel name
+ * output      - none
+ * side effects - error message sent to client
+ */
+static void
+send_join_error(struct Client *source_p, int numeric, const char *name)
+{
+       /* This stuff is necessary because the form_str macro only
+        * accepts constants.
+        */
+       switch (numeric)
+       {
+#define NORMAL_NUMERIC(i)                                              \
+               case i:                                                 \
+                       sendto_one(source_p, form_str(i),               \
+                                       me.name, source_p->name, name); \
+                       break
+
+               NORMAL_NUMERIC(ERR_BANNEDFROMCHAN);
+               NORMAL_NUMERIC(ERR_INVITEONLYCHAN);
+               NORMAL_NUMERIC(ERR_BADCHANNELKEY);
+               NORMAL_NUMERIC(ERR_CHANNELISFULL);
+               NORMAL_NUMERIC(ERR_NEEDREGGEDNICK);
+               NORMAL_NUMERIC(ERR_THROTTLE);
+
+               default:
+                       sendto_one_numeric(source_p, numeric,
+                                       "%s :Cannot join channel", name);
+                       break;
+       }
 }
 
 static void
@@ -1076,7 +1091,7 @@ set_final_mode(struct Mode *mode, struct Mode *oldmode)
                        dir = MODE_DEL;
                }
                *mbuf++ = 'k';
-               len = rb_sprintf(pbuf, "%s ", oldmode->key);
+               len = sprintf(pbuf, "%s ", oldmode->key);
                pbuf += len;
        }
        if(oldmode->join_num && !mode->join_num)
@@ -1105,7 +1120,7 @@ set_final_mode(struct Mode *mode, struct Mode *oldmode)
                        dir = MODE_ADD;
                }
                *mbuf++ = 'l';
-               len = rb_sprintf(pbuf, "%d ", mode->limit);
+               len = sprintf(pbuf, "%d ", mode->limit);
                pbuf += len;
        }
        if(mode->key[0] && strcmp(oldmode->key, mode->key))
@@ -1116,7 +1131,7 @@ set_final_mode(struct Mode *mode, struct Mode *oldmode)
                        dir = MODE_ADD;
                }
                *mbuf++ = 'k';
-               len = rb_sprintf(pbuf, "%s ", mode->key);
+               len = sprintf(pbuf, "%s ", mode->key);
                pbuf += len;
        }
        if(mode->join_num && (oldmode->join_num != mode->join_num || oldmode->join_time != mode->join_time))
@@ -1127,7 +1142,7 @@ set_final_mode(struct Mode *mode, struct Mode *oldmode)
                        dir = MODE_ADD;
                }
                *mbuf++ = 'j';
-               len = rb_sprintf(pbuf, "%d:%d ", mode->join_num, mode->join_time);
+               len = sprintf(pbuf, "%d:%d ", mode->join_num, mode->join_time);
                pbuf += len;
        }
        if(mode->forward[0] && strcmp(oldmode->forward, mode->forward) &&
@@ -1139,7 +1154,7 @@ set_final_mode(struct Mode *mode, struct Mode *oldmode)
                        dir = MODE_ADD;
                }
                *mbuf++ = 'f';
-               len = rb_sprintf(pbuf, "%s ", mode->forward);
+               len = sprintf(pbuf, "%s ", mode->forward);
                pbuf += len;
        }
        *mbuf = '\0';
@@ -1149,8 +1164,8 @@ set_final_mode(struct Mode *mode, struct Mode *oldmode)
  * remove_our_modes
  *
  * inputs      -
- * output      - 
- * side effects        - 
+ * output      -
+ * side effects        -
  */
 static void
 remove_our_modes(struct Channel *chptr, struct Client *source_p)
@@ -1264,7 +1279,7 @@ remove_ban_list(struct Channel *chptr, struct Client *source_p,
 
        pbuf = lparabuf;
 
-       cur_len = mlen = rb_sprintf(lmodebuf, ":%s MODE %s -", source_p->name, chptr->chname);
+       cur_len = mlen = sprintf(lmodebuf, ":%s MODE %s -", source_p->name, chptr->chname);
        mbuf = lmodebuf + mlen;
 
        RB_DLINK_FOREACH_SAFE(ptr, next_ptr, list->head)
@@ -1292,9 +1307,9 @@ remove_ban_list(struct Channel *chptr, struct Client *source_p,
                *mbuf++ = c;
                cur_len += plen;
                if (banptr->forward)
-                       pbuf += rb_sprintf(pbuf, "%s$%s ", banptr->banstr, banptr->forward);
+                       pbuf += sprintf(pbuf, "%s$%s ", banptr->banstr, banptr->forward);
                else
-                       pbuf += rb_sprintf(pbuf, "%s ", banptr->banstr);
+                       pbuf += sprintf(pbuf, "%s ", banptr->banstr);
                count++;
 
                free_ban(banptr);