add checkoutput.patch - made a few changes

[irc/quakenet/snircd-patchqueue.git] / accountcollision.patch

Add account collision kills, controlled by feature ACCOUNT_COLLISION_KILLS

include/ircd_features.h
ircd/ircd_features.c
  add new feature

include/numeric.h
ircd/s_err.c
  add new numerc 435 ERR_ACCOUNTCOLLISION (mirrored after 436 ERR_NICKCOLLISION)

ircd/m_account.c
  add the account collision kills

diff -r 4599c781548c include/ircd_features.h
--- a/include/ircd_features.h	Tue Jan 27 18:15:32 2009 +0100
+++ b/include/ircd_features.h	Tue Jan 27 18:19:05 2009 +0100
@@ -102,6 +102,7 @@
   FEAT_AUTH_TIMEOUT,
   FEAT_ANNOUNCE_INVITES,
   FEAT_WELCOME,
+  FEAT_ACCOUNT_COLLISION_KILLS,
 
   /* features that affect all operators */
   FEAT_EXTENDED_CHECKCMD,
diff -r 4599c781548c include/numeric.h
--- a/include/numeric.h	Tue Jan 27 18:15:32 2009 +0100
+++ b/include/numeric.h	Tue Jan 27 18:19:05 2009 +0100
@@ -358,6 +358,7 @@
 /*	ERR_NORULES	     434   unreal */
 /*      ERR_SERVICECONFUSED  435 ? */
 /*	ERR_BANONCHAN	     435   dalnet */
+#define ERR_ACCOUNTCOLLISION 435        /* QuakeNet extension */
 #define ERR_NICKCOLLISION    436
 #define ERR_BANNICKCHANGE    437        /* Undernet extension */
      /* ERR_UNAVAILRESOURCE  437           IRCnet extension */
diff -r 4599c781548c ircd/ircd_features.c
--- a/ircd/ircd_features.c	Tue Jan 27 18:15:32 2009 +0100
+++ b/ircd/ircd_features.c	Tue Jan 27 18:19:05 2009 +0100
@@ -356,6 +356,7 @@
   F_I(AUTH_TIMEOUT, 0, 9, 0),
   F_B(ANNOUNCE_INVITES, 0, 0, 0),
   F_B(WELCOME, 0, 1, 0),
+  F_B(ACCOUNT_COLLISION_KILLS, 0, 0, 0),
 
   /* features that affect all operators */
   F_B(EXTENDED_CHECKCMD, 0, 0, 0),
diff -r 4599c781548c ircd/m_account.c
--- a/ircd/m_account.c	Tue Jan 27 18:15:32 2009 +0100
+++ b/ircd/m_account.c	Tue Jan 27 18:19:05 2009 +0100
@@ -82,10 +82,12 @@
 
 #include "client.h"
 #include "ircd.h"
+#include "ircd_features.h"
 #include "ircd_log.h"
 #include "ircd_reply.h"
 #include "ircd_string.h"
 #include "msg.h"
+#include "numeric.h"
 #include "numnicks.h"
 #include "s_debug.h"
 #include "s_user.h"
@@ -138,10 +140,45 @@
   if (IsAccount(acptr)) {
     if (strcmp(cli_user(acptr)->account, parv[2]) || 
         (cli_user(acptr)->acc_create != acc_create) ||
-        (cli_user(acptr)->acc_id != acc_id))
-      return protocol_violation(cptr, "ACCOUNT for already registered user %s "
-				"(%s -> %s)", cli_name(acptr),
-				cli_user(acptr)->account, parv[2]);
+        (cli_user(acptr)->acc_id != acc_id)) {
+
+      /* license to KILL for account collisions */
+      /* TODO: what if just the timestamp differs?
+       *       account and ID are the same
+       *       could we not resolve the conflict without KILL?
+       *       ie. oldest timestamp overrules?
+       */
+      if (feature_bool(FEAT_ACCOUNT_COLLISION_KILLS)) {
+        /* inform ops */
+        if (strcmp(cli_user(acptr)->account, parv[2]))
+          sendto_opmask_butone(0, SNO_OLDSNO,
+            "Account collision from %C on %C (%s <- %C %s (Different account))",
+            sptr, acptr, cli_user(acptr)->account, cptr, parv[2]);
+        else if (cli_user(acptr)->acc_id != acc_id)
+          sendto_opmask_butone(0, SNO_OLDSNO,
+            "Account collision from %C on %C (%lu <- %C %lu (Different ID))",
+            sptr, acptr,cli_user(acptr)->acc_id, cptr, acc_id);
+        else
+          sendto_opmask_butone(0, SNO_OLDSNO,
+            "Account collision from %C on %C (%Tu <- %C %Tu (Different timestamp))",
+            sptr, acptr, cli_user(acptr)->acc_create, cptr, acc_create);
+
+        /* tell victim why we kill them */
+        /* TODO: send the account we got here (parv[2]) or just a *? */
+        send_reply(acptr, ERR_ACCOUNTCOLLISION, "*");
+        /* Inform the rest of the net... */
+        ServerStats->is_kill++;
+        sendcmdto_serv_butone(&me, CMD_KILL, 0, "%C :%s (account collision)",
+          acptr, cli_name(&me));
+        /* don't go sending off a QUIT message... */
+        SetFlag(acptr, FLAG_KILLED);
+        /* remove them locally. */
+        return exit_client_msg(cptr, acptr, &me, "Killed (%s (account collision))",
+          feature_str(FEAT_HIS_SERVERNAME));
+      } else
+        return protocol_violation(cptr, "ACCOUNT for already registered user %s "
+          "(%s -> %s)", cli_name(acptr), cli_user(acptr)->account, parv[2]);
+	 }
 
     cli_user(acptr)->acc_flags = acc_flags;
 
diff -r 4599c781548c ircd/s_err.c
--- a/ircd/s_err.c	Tue Jan 27 18:15:32 2009 +0100
+++ b/ircd/s_err.c	Tue Jan 27 18:19:05 2009 +0100
@@ -902,7 +902,7 @@
 /* 434 */
   { 0 },
 /* 435 */
-  { 0 },
+  { ERR_ACCOUNTCOLLISION, "%s :Account collision KILL", "435" },
 /* 436 */
   { ERR_NICKCOLLISION, "%s :Nickname collision KILL", "436" },
 /* 437 */