]> arthur.barton.de Git - ngircd-alex.git/blobdiff - src/ngircd/irc-op.c
Make sure that the target user is able to join a local channel
[ngircd-alex.git] / src / ngircd / irc-op.c
index 5c12faf78022ac8929d239029a4e7d2e8ff2fe47..2e5c680aaa52ddde65b6a0ade839e90e96770cdb 100644 (file)
@@ -1,6 +1,6 @@
 /*
  * ngIRCd -- The Next Generation IRC Daemon
- * Copyright (c)2001-2005 by Alexander Barton (alex@barton.de)
+ * Copyright (c)2001-2014 Alexander Barton (alex@barton.de) and Contributors.
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
  * Channel operator commands
  */
 
-#include "imp.h"
 #include <assert.h>
 #include <string.h>
-#include <stdio.h>
 
-#include "defines.h"
 #include "conn.h"
 #include "channel.h"
+#include "irc-macros.h"
 #include "irc-write.h"
 #include "lists.h"
 #include "log.h"
 #include "messages.h"
 #include "parse.h"
 
-#include "exp.h"
 #include "irc-op.h"
 
+/* Local functions */
 
 static bool
 try_kick(CLIENT *peer, CLIENT* from, const char *nick, const char *channel,
@@ -41,13 +39,22 @@ try_kick(CLIENT *peer, CLIENT* from, const char *nick, const char *channel,
        CLIENT *target = Client_Search(nick);
 
        if (!target)
-               return IRC_WriteStrClient(from, ERR_NOSUCHNICK_MSG, Client_ID(from), nick);
+               return IRC_WriteErrClient(from, ERR_NOSUCHNICK_MSG,
+                                         Client_ID(from), nick);
 
        Channel_Kick(peer, target, from, channel, reason);
        return true;
 }
 
+/* Global functions */
 
+/**
+ * Handler for the IRC command "KICK".
+ *
+ * @param Client The client from which this command has been received.
+ * @param Req Request structure with prefix and all parameters.
+ * @return CONNECTED or DISCONNECTED.
+ */
 GLOBAL bool
 IRC_KICK(CLIENT *Client, REQUEST *Req)
 {
@@ -60,9 +67,7 @@ IRC_KICK(CLIENT *Client, REQUEST *Req)
        assert( Client != NULL );
        assert( Req != NULL );
 
-       if ((Req->argc < 2) || (Req->argc > 3))
-               return IRC_WriteStrClient(Client, ERR_NEEDMOREPARAMS_MSG,
-                                       Client_ID(Client), Req->command);
+       _IRC_GET_SENDER_OR_RETURN_(from, Req, Client)
 
        while (*itemList) {
                if (*itemList == ',') {
@@ -81,15 +86,6 @@ IRC_KICK(CLIENT *Client, REQUEST *Req)
                itemList++;
        }
 
-       if (Client_Type(Client) == CLIENT_SERVER)
-               from = Client_Search(Req->prefix);
-       else
-               from = Client;
-
-       if (!from)
-               return IRC_WriteStrClient(Client, ERR_NOSUCHNICK_MSG,
-                                       Client_ID(Client), Req->prefix);
-
        reason = Req->argc == 3 ? Req->argv[2] : Client_ID(from);
        currentNick = Req->argv[1];
        currentChannel = Req->argv[0];
@@ -122,13 +118,19 @@ IRC_KICK(CLIENT *Client, REQUEST *Req)
                        nickCount--;
                }
        } else {
-               return IRC_WriteStrClient(Client, ERR_NEEDMOREPARAMS_MSG,
+               return IRC_WriteErrClient(Client, ERR_NEEDMOREPARAMS_MSG,
                                        Client_ID(Client), Req->command);
        }
        return true;
 } /* IRC_KICK */
 
-
+/**
+ * Handler for the IRC command "INVITE".
+ *
+ * @param Client The client from which this command has been received.
+ * @param Req Request structure with prefix and all parameters.
+ * @return CONNECTED or DISCONNECTED.
+ */
 GLOBAL bool
 IRC_INVITE(CLIENT *Client, REQUEST *Req)
 {
@@ -140,51 +142,56 @@ IRC_INVITE(CLIENT *Client, REQUEST *Req)
        assert( Client != NULL );
        assert( Req != NULL );
 
-       if (Req->argc != 2)
-               return IRC_WriteStrClient(Client, ERR_NEEDMOREPARAMS_MSG,
-                                       Client_ID(Client), Req->command);
-
-       if (Client_Type(Client) == CLIENT_SERVER)
-               from = Client_Search(Req->prefix);
-       else
-               from = Client;
-       if (!from)
-               return IRC_WriteStrClient(Client, ERR_NOSUCHNICK_MSG,
-                                       Client_ID(Client), Req->prefix);
+       _IRC_GET_SENDER_OR_RETURN_(from, Req, Client)
 
        /* Search user */
        target = Client_Search(Req->argv[0]);
        if (!target || (Client_Type(target) != CLIENT_USER))
-               return IRC_WriteStrClient(from, ERR_NOSUCHNICK_MSG,
-                               Client_ID(Client), Req->argv[0]);
+               return IRC_WriteErrClient(from, ERR_NOSUCHNICK_MSG,
+                                         Client_ID(Client), Req->argv[0]);
+
+       if (Req->argv[1][0] == '&') {
+               /* Local channel. Make sure the target user is on this server! */
+               if (Client_Conn(target) == NONE)
+                       return IRC_WriteErrClient(from, ERR_USERNOTONSERV_MSG,
+                                                 Client_ID(Client),
+                                                 Req->argv[0]);
+       }
 
        chan = Channel_Search(Req->argv[1]);
        if (chan) {
                /* Channel exists. Is the user a valid member of the channel? */
                if (!Channel_IsMemberOf(chan, from))
-                       return IRC_WriteStrClient(from, ERR_NOTONCHANNEL_MSG, Client_ID(Client), Req->argv[1]);
+                       return IRC_WriteErrClient(from, ERR_NOTONCHANNEL_MSG,
+                                                 Client_ID(Client),
+                                                 Req->argv[1]);
 
                /* Is the channel "invite-disallow"? */
-               if (strchr(Channel_Modes(chan), 'V'))
-                       return IRC_WriteStrClient(from, ERR_NOINVITE_MSG,
-                               Client_ID(from), Channel_Name(chan));
+               if (Channel_HasMode(chan, 'V'))
+                       return IRC_WriteErrClient(from, ERR_NOINVITE_MSG,
+                                                 Client_ID(from),
+                                                 Channel_Name(chan));
 
                /* Is the channel "invite-only"? */
-               if (strchr(Channel_Modes(chan), 'i')) {
-                       /* Yes. The user must be channel owner/admin/operator/halfop! */
-                       if (!strchr(Channel_UserModes(chan, from), 'q') &&
-                           !strchr(Channel_UserModes(chan, from), 'a') &&
-                           !strchr(Channel_UserModes(chan, from), 'o') &&
-                           !strchr(Channel_UserModes(chan, from), 'h'))
-                               return IRC_WriteStrClient(from, ERR_CHANOPRIVSNEEDED_MSG,
-                                               Client_ID(from), Channel_Name(chan));
+               if (Channel_HasMode(chan, 'i')) {
+                       /* Yes. The user issuing the INVITE command must be
+                        * channel owner/admin/operator/halfop! */
+                       if (!Channel_UserHasMode(chan, from, 'q') &&
+                           !Channel_UserHasMode(chan, from, 'a') &&
+                           !Channel_UserHasMode(chan, from, 'o') &&
+                           !Channel_UserHasMode(chan, from, 'h'))
+                               return IRC_WriteErrClient(from,
+                                                         ERR_CHANOPRIVSNEEDED_MSG,
+                                                         Client_ID(from),
+                                                         Channel_Name(chan));
                        remember = true;
                }
 
                /* Is the target user already member of the channel? */
                if (Channel_IsMemberOf(chan, target))
-                       return IRC_WriteStrClient(from, ERR_USERONCHANNEL_MSG,
-                                       Client_ID(from), Req->argv[0], Req->argv[1]);
+                       return IRC_WriteErrClient(from, ERR_USERONCHANNEL_MSG,
+                                                 Client_ID(from),
+                                                 Req->argv[0], Req->argv[1]);
 
                /* If the target user is banned on that channel: remember invite */
                if (Lists_Check(Channel_GetListBans(chan), target))
@@ -192,33 +199,35 @@ IRC_INVITE(CLIENT *Client, REQUEST *Req)
 
                if (remember) {
                        /* We must remember this invite */
-                       if (!Channel_AddInvite(chan, Client_Mask(target), true))
+                       if (!Channel_AddInvite(chan, Client_MaskCloaked(target),
+                                               true))
                                return CONNECTED;
                }
        }
 
-       LogDebug("User \"%s\" invites \"%s\" to \"%s\" ...", Client_Mask(from), Req->argv[0], Req->argv[1]);
-
+       LogDebug("User \"%s\" invites \"%s\" to \"%s\" ...", Client_Mask(from),
+                Req->argv[0], Req->argv[1]);
 
        /*
-        * RFC 2812 says:
-        * 'There is no requirement that the channel [..] must exist or be a valid channel'
-        * The problem with this is that this allows the "channel" to contain spaces,
-        * in which case we must prefix its name with a colon to make it clear that
-        * it is only a single argument.
+        * RFC 2812 states:
+        * 'There is no requirement that the channel [..] must exist or be a
+        * valid channel'. The problem with this is that this allows the
+        * "channel" to contain spaces, in which case we must prefix its name
+        * with a colon to make it clear that it is only a single argument.
         */
        colon_if_necessary = strchr(Req->argv[1], ' ') ? ":":"";
        /* Inform target client */
        IRC_WriteStrClientPrefix(target, from, "INVITE %s %s%s", Req->argv[0],
-                                       colon_if_necessary, Req->argv[1]);
+                                 colon_if_necessary, Req->argv[1]);
 
        if (Client_Conn(target) > NONE) {
                /* The target user is local, so we have to send the status code */
                if (!IRC_WriteStrClientPrefix(from, target, RPL_INVITING_MSG,
-                       Client_ID(from), Req->argv[0], colon_if_necessary, Req->argv[1]))
+                                              Client_ID(from), Req->argv[0],
+                                              colon_if_necessary, Req->argv[1]))
                        return DISCONNECTED;
 
-               if (strchr(Client_Modes(target), 'a') &&
+               if (Client_HasMode(target, 'a') &&
                        !IRC_WriteStrClient(from, RPL_AWAY_MSG, Client_ID(from),
                                        Client_ID(target), Client_Away(target)))
                                return DISCONNECTED;
@@ -226,5 +235,4 @@ IRC_INVITE(CLIENT *Client, REQUEST *Req)
        return CONNECTED;
 } /* IRC_INVITE */
 
-
 /* -eof- */