Rework check for number of parameters

[ngircd-alex.git] / src / ngircd / op.c

diff --git a/src/ngircd/op.c b/src/ngircd/op.c
index 64c7e1e1fb2142e0bd6f3c5850220c669a422756..2d36cd559196c3e7639367370bf6960896f6e451 100644 (file)
--- a/src/ngircd/op.c
+++ b/src/ngircd/op.c
@@ -1,6 +1,6 @@
 /*
  * ngIRCd -- The Next Generation IRC Daemon
- * Copyright (c)2001-2008 Alexander Barton (alex@barton.de)
+ * Copyright (c)2001-2013 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
@@ -9,7 +9,6 @@
  * Please read the file COPYING, README and AUTHORS for more information.
  */
 
-
 #include "portab.h"
 
 /**
@@ -32,6 +31,7 @@
 #include <exp.h>
 #include "op.h"
 
+
 /**
  * Return and log a "no privileges" message.
  */
@@ -46,21 +46,27 @@ Op_NoPrivileges(CLIENT * Client, REQUEST * Req)
        if (from) {
                Log(LOG_NOTICE, "No privileges: client \"%s\" (%s), command \"%s\"",
                    Req->prefix, Client_Mask(Client), Req->command);
-               return IRC_WriteStrClient(from, ERR_NOPRIVILEGES_MSG,
+               return IRC_WriteErrClient(from, ERR_NOPRIVILEGES_MSG,
                                          Client_ID(from));
        } else {
                Log(LOG_NOTICE, "No privileges: client \"%s\", command \"%s\"",
                    Client_Mask(Client), Req->command);
-               return IRC_WriteStrClient(Client, ERR_NOPRIVILEGES_MSG,
+               return IRC_WriteErrClient(Client, ERR_NOPRIVILEGES_MSG,
                                          Client_ID(Client));
        }
 } /* Op_NoPrivileges */
 
 
 /**
- * Check that the client is an IRC operator allowed to administer this server.
+ * Check that the originator of a request is an IRC operator and allowed
+ * to administer this server.
+ *
+ * @param CLient Client from which the command has been received.
+ * @param Req Request structure.
+ * @return CLIENT structure of the client that initiated the command or
+ *        NULL if client is not allowed to execute operator commands.
  */
-GLOBAL bool
+GLOBAL CLIENT *
 Op_Check(CLIENT * Client, REQUEST * Req)
 {
        CLIENT *c;
@@ -72,13 +78,21 @@ Op_Check(CLIENT * Client, REQUEST * Req)
                c = Client_Search(Req->prefix);
        else
                c = Client;
+
        if (!c)
-               return false;
+               return NULL;
+       if (Client_Type(Client) == CLIENT_SERVER
+           && Client_Type(c) == CLIENT_SERVER)
+               return c;
        if (!Client_HasMode(c, 'o'))
-               return false;
+               return NULL;
        if (!Client_OperByMe(c) && !Conf_AllowRemoteOper)
-               return false;
+               return NULL;
+
        /* The client is an local IRC operator, or this server is configured
         * to trust remote operators. */
-       return true;
+       return c;
 } /* Op_Check */
+
+
+/* -eof- */