From d19b4e70ad48efe6de266cd20bb829889e693d1d Mon Sep 17 00:00:00 2001
From: Bram Matthys <syzop@vulnscan.org>
Date: Sat, 24 Aug 2019 19:45:26 +0200
Subject: [PATCH] Remove old and broken option SHOW_INVISIBLE_LUSERS

---
 include/config.h    | 12 ------------
 src/debug.c         |  3 ---
 src/modules/trace.c |  6 ------
 3 files changed, 21 deletions(-)

diff --git a/include/config.h b/include/config.h
index 673a3bf23..be6729af4 100644
--- a/include/config.h
+++ b/include/config.h
@@ -88,9 +88,6 @@
 
 #define HOW_MANY_FREELINKS_ALLOWED 	200	/* default: 200 */
 
-/* NOTE: On some systems, valloc() causes many problems. */
-#undef	VALLOC			/* Define this if you have valloc(3) */
-
 /*
  * read/write are restarted after signals defining this 1, gets
  * siginterrupt call compiled, which attempts to remove this
@@ -122,15 +119,6 @@
 #define BPATH		CONFDIR"/bot.motd"	/* Bot MOTD */
 #define IRCDTUNE 	PERMDATADIR"/ircd.tune"	/* tuning .. */
 
-/* SHOW_INVISIBLE_LUSERS
- *
- * As defined this will show the correct invisible count for anyone who does
- * LUSERS on your server. On a large net this doesnt mean much, but on a
- * small net it might be an advantage to undefine it.
- * (This will get defined for you if you're using userload (stats w).  -mlv)
- */
-#define	SHOW_INVISIBLE_LUSERS
-
 /** FAKELAG_CONFIGURABLE makes it possible to make certain classes exempted
  * from 'fake lag' (that is, the artificial delay that is added by the ircd
  * to prevent flooding, which causes the messages/commands of the user to
diff --git a/src/debug.c b/src/debug.c
index 5ab4ef0d5..2f45b4000 100644
--- a/src/debug.c
+++ b/src/debug.c
@@ -30,9 +30,6 @@ MODVAR char serveropts[] = {
 	'F',
 	/* Hub (always) */
 	'h',
-#ifdef	SHOW_INVISIBLE_LUSERS
-	'i',
-#endif
 	/* NOSPOOF (always) */
 	'n',
 #ifdef	VALLOC
diff --git a/src/modules/trace.c b/src/modules/trace.c
index 1facac306..e19189412 100644
--- a/src/modules/trace.c
+++ b/src/modules/trace.c
@@ -126,14 +126,8 @@ CMD_FUNC(m_trace)
 		{
 			if (acptr->from->fd < 0)
 				continue;
-#ifdef	SHOW_INVISIBLE_LUSERS
 			if (IsPerson(acptr))
 				link_u[acptr->from->fd]++;
-#else
-			if (IsPerson(acptr) &&
-			    (!IsInvisible(acptr) || ValidatePermissionsForPath("client:see:trace:invisible-users",sptr,acptr,NULL,NULL)))
-				link_u[acptr->from->fd]++;
-#endif
 			else if (IsServer(acptr))
 				link_s[acptr->from->fd]++;
 		}