Fixes for config reader

git-svn-id: http://svn.inspircd.org/repository/trunk/inspircd@11504 e03df62e-2008-0410-955e-edbf42e46eb7

3 files changed, 205 insertions, 397 deletions

diff --git a/include/configreader.h b/include/configreader.h
index c4b603342..b5e6a887b 100644
--- a/include/configreader.h
+++ b/include/configreader.h
@@ -32,195 +32,10 @@
 
 /* Required forward definitions */
 class ServerConfig;
+class ServerLimits;
 class InspIRCd;
 class BufferedSocket;
 
-/** Types of data in the core config
- */
-enum ConfigDataType
-{
-	DT_NOTHING       = 0,		/* No data */
-	DT_INTEGER       = 1,		/* Integer */
-	DT_CHARPTR       = 2,		/* Char pointer */
-	DT_BOOLEAN       = 3,		/* Boolean */
-	DT_HOSTNAME	 = 4,		/* Hostname syntax */
-	DT_NOSPACES	 = 5,		/* No spaces */
-	DT_IPADDRESS	 = 6,		/* IP address (v4, v6) */
-	DT_CHANNEL	 = 7,		/* Channel name */
-	DT_ALLOW_WILD	 = 64,		/* Allow wildcards/CIDR in DT_IPADDRESS */
-	DT_ALLOW_NEWLINE = 128		/* New line characters allowed in DT_CHARPTR */
-};
-
-/** The maximum number of values in a core configuration tag. Can be increased if needed.
- */
-#define MAX_VALUES_PER_TAG 18
-
-/** Holds a config value, either string, integer or boolean.
- * Callback functions receive one or more of these, either on
- * their own as a reference, or in a reference to a deque of them.
- * The callback function can then alter the values of the ValueItem
- * classes to validate the settings.
- */
-class ValueItem
-{
-	/** Actual data */
-	std::string v;
- public:
-	/** Initialize with an int */
-	ValueItem(int value);
-	/** Initialize with a bool */
-	ValueItem(bool value);
-	/** Initialize with a char pointer */
-	ValueItem(const char* value);
-	/** Change value to a char pointer */
-	void Set(const char* val);
-	/** Change value to an int */
-	void Set(int value);
-	/** Get value as an int */
-	int GetInteger();
-	/** Get value as a string */
-	char* GetString();
-	/** Get value as a bool */
-	bool GetBool();
-};
-
-/** The base class of the container 'ValueContainer'
- * used internally by the core to hold core values.
- */
-class ValueContainerBase
-{
- public:
-	/** Constructor */
-	ValueContainerBase() { }
-	/** Destructor */
-	virtual ~ValueContainerBase() { }
-};
-
-/** ValueContainer is used to contain pointers to different
- * core values such as the server name, maximum number of
- * clients etc.
- * It is specialized to hold a data type, then pointed at
- * a value in the ServerConfig class. When the value has been
- * read and validated, the Set method is called to write the
- * value safely in a type-safe manner.
- */
-template<typename T> class ValueContainer : public ValueContainerBase
-{
-	/** Contained item */
-	T val;
- public:
-
-	/** Initialize with nothing */
-	ValueContainer()
-	{
-		val = NULL;
-	}
-
-	/** Initialize with a value of type T */
-	ValueContainer(T Val)
-	{
-		val = Val;
-	}
-
-	/** Change value to type T of size s */
-	void Set(T newval, size_t s)
-	{
-		memcpy(val, newval, s);
-	}
-};
-
-/** A specialization of ValueContainer to hold a pointer to a bool
- */
-typedef ValueContainer<bool*> ValueContainerBool;
-
-/** A specialization of ValueContainer to hold a pointer to
- * an unsigned int
- */
-typedef ValueContainer<unsigned int*> ValueContainerUInt;
-
-/** A specialization of ValueContainer to hold a pointer to
- * a char array.
- */
-typedef ValueContainer<char*> ValueContainerChar;
-
-/** A specialization of ValueContainer to hold a pointer to
- * an int
- */
-typedef ValueContainer<int*> ValueContainerInt;
-
-/** A specialization of ValueContainer to hold a pointer to
- * a size_t.
- */
-typedef ValueContainer<size_t*> ValueContainerST;
-
-/** A set of ValueItems used by multi-value validator functions
- */
-typedef std::deque<ValueItem> ValueList;
-
-/** A callback for validating a single value
- */
-typedef bool (*Validator)(ServerConfig* conf, const char*, const char*, ValueItem&);
-/** A callback for validating multiple value entries
- */
-typedef bool (*MultiValidator)(ServerConfig* conf, const char*, char**, ValueList&, int*);
-/** A callback indicating the end of a group of entries
- */
-typedef bool (*MultiNotify)(ServerConfig* conf, const char*);
-
-/** Holds a core configuration item and its callbacks
- */
-struct InitialConfig
-{
-	/** Tag name */
-	const char* tag;
-	/** Value name */
-	const char* value;
-	/** Default, if not defined */
-	const char* default_value;
-	/** Value containers */
-	ValueContainerBase* val;
-	/** Data types */
-	int datatype;
-	/** Validation function */
-	Validator validation_function;
-};
-
-/** Represents a deprecated configuration tag.
- */
-struct Deprecated
-{
-	/** Tag name
-	 */
-	const char* tag;
-	/** Tag value
-	 */
-	const char* value;
-	/** Reason for deprecation
-	 */
-	const char* reason;
-};
-
-/** Holds a core configuration item and its callbacks
- * where there may be more than one item
- */
-struct MultiConfig
-{
-	/** Tag name */
-	const char*	tag;
-	/** One or more items within tag */
-	const char*	items[MAX_VALUES_PER_TAG];
-	/** One or more defaults for items within tags */
-	const char* items_default[MAX_VALUES_PER_TAG];
-	/** One or more data types */
-	int		datatype[MAX_VALUES_PER_TAG];
-	/** Initialization function */
-	MultiNotify	init_function;
-	/** Validation function */
-	MultiValidator	validation_function;
-	/** Completion function */
-	MultiNotify	finish_function;
-};
-
 /** A set of oper types
  */
 typedef std::map<irc::string,std::string> opertype_t;
@@ -318,6 +133,9 @@ class CoreExport ServerConfig : public Extensible
 	 */
 	std::vector<std::string> include_stack;
 
+	/* classes removed by this rehash */
+	std::vector<ConnectClass*> removed_classes;
+
 	/** This private method processes one line of
 	 * configutation, appending errors to errorstream
 	 * and setting error if an error has occured.
@@ -328,6 +146,9 @@ class CoreExport ServerConfig : public Extensible
 	 */
 	bool CheckOnce(const char* tag);
 
+	void CrossCheckOperClassType();
+	void CrossCheckConnectBlocks(ServerConfig* current);
+
  public:
 	/** Process an include executable directive
 	 */
@@ -351,9 +172,6 @@ class CoreExport ServerConfig : public Extensible
 	/** Used to indicate who we announce invites to on a channel */
 	enum InviteAnnounceState { INVITE_ANNOUNCE_NONE, INVITE_ANNOUNCE_ALL, INVITE_ANNOUNCE_OPS, INVITE_ANNOUNCE_DYNAMIC };
 
-	/** Pointer to function that validates dns server addresses (can be changed depending on platform) */
-	Validator DNSServerValidator;
-
 	/** Returns the creator InspIRCd pointer
 	 */
 	InspIRCd* GetInstance();
@@ -513,12 +331,12 @@ class CoreExport ServerConfig : public Extensible
 	 * overridden in the configuration file via
 	 * the <options> tag.
 	 */
-	char ModPath[1024];
+	std::string ModPath;
 
 	/** The full pathname to the executable, as
 	 * given in argv[0] when the program starts.
 	 */
-	char MyExecutable[1024];
+	std::string MyExecutable;
 
 	/** The file handle of the logfile. If this
 	 * value is NULL, the log file is not open,
@@ -658,7 +476,7 @@ class CoreExport ServerConfig : public Extensible
 	/** The full pathname and filename of the PID
 	 * file as defined in the configuration.
 	 */
-	char PID[1024];
+	std::string PID;
 
 	/** The connect classes in use by the IRC server.
 	 */
@@ -766,10 +584,6 @@ class CoreExport ServerConfig : public Extensible
 	 */
 	ServerConfig(InspIRCd* Instance);
 
-	/** Clears the include stack in preperation for a Read() call.
-	 */
-	void ClearStack();
-
 	/** Get server ID as string with required leading zeroes
 	 */
 	std::string GetSID();
@@ -791,6 +605,7 @@ class CoreExport ServerConfig : public Extensible
 	/** Apply configuration changes from the old configuration.
 	 */
 	void Apply(ServerConfig* old, const std::string &useruid);
+	void ApplyModules(User* user);
 
 	/** Read a file into a file_cache object
 	 */
@@ -872,17 +687,7 @@ class CoreExport ServerConfig : public Extensible
 	 */
 	int ConfVarEnum(const std::string &tag, int index);
 
-	/** Validates a hostname value, throwing ConfigException if it is not valid
-	 */
-	void ValidateHostname(const char* p, const std::string &tag, const std::string &val);
-
-	/** Validates an IP address value, throwing ConfigException if it is not valid
-	 */
-	void ValidateIP(const char* p, const std::string &tag, const std::string &val, bool wild);
-
-	/** Validates a value that should not contain spaces, throwing ConfigException of it is not valid
-	 */
-	void ValidateNoSpaces(const char* p, const std::string &tag, const std::string &val);
+	bool ApplyDisabledCommands(const char* data);
 
 	/** Returns the fully qualified path to the inspircd directory
 	 * @return The full program directory
@@ -907,50 +712,202 @@ class CoreExport ServerConfig : public Extensible
 
 };
 
-/** Initialize the disabled commands list
+
+/** Types of data in the core config
+ */
+enum ConfigDataType
+{
+	DT_NOTHING       = 0,		/* No data */
+	DT_INTEGER       = 1,		/* Integer */
+	DT_CHARPTR       = 2,		/* Char pointer */
+	DT_BOOLEAN       = 3,		/* Boolean */
+	DT_HOSTNAME	 = 4,		/* Hostname syntax */
+	DT_NOSPACES	 = 5,		/* No spaces */
+	DT_IPADDRESS	 = 6,		/* IP address (v4, v6) */
+	DT_CHANNEL	 = 7,		/* Channel name */
+	DT_LIMIT     = 8,       /* size_t */
+	DT_ALLOW_WILD	 = 64,		/* Allow wildcards/CIDR in DT_IPADDRESS */
+	DT_ALLOW_NEWLINE = 128		/* New line characters allowed in DT_CHARPTR */
+};
+
+/** The maximum number of values in a core configuration tag. Can be increased if needed.
+ */
+#define MAX_VALUES_PER_TAG 18
+
+/** Holds a config value, either string, integer or boolean.
+ * Callback functions receive one or more of these, either on
+ * their own as a reference, or in a reference to a deque of them.
+ * The callback function can then alter the values of the ValueItem
+ * classes to validate the settings.
  */
-CoreExport bool InitializeDisabledCommands(const char* data, InspIRCd* ServerInstance);
+class ValueItem
+{
+	/** Actual data */
+	std::string v;
+ public:
+	/** Initialize with an int */
+	ValueItem(int value);
+	/** Initialize with a bool */
+	ValueItem(bool value);
+	/** Initialize with a string */
+	ValueItem(const char* value) : v(value) { }
+	/** Change value to a string */
+	void Set(const std::string &val);
+	/** Change value to an int */
+	void Set(int value);
+	/** Get value as an int */
+	int GetInteger();
+	/** Get value as a string */
+	const char* GetString() const;
+	/** Get value as a string */
+	inline const std::string& GetValue() const { return v; }
+	/** Get value as a bool */
+	bool GetBool();
+};
 
-/** Initialize the oper types
+/** The base class of the container 'ValueContainer'
+ * used internally by the core to hold core values.
  */
-bool InitTypes(ServerConfig* conf, const char* tag);
+class ValueContainerBase
+{
+ public:
+	/** Constructor */
+	ValueContainerBase() { }
+	/** Destructor */
+	virtual ~ValueContainerBase() { }
+};
 
-/** Initialize the oper classes
+/** ValueContainer is used to contain pointers to different
+ * core values such as the server name, maximum number of
+ * clients etc.
+ * It is specialized to hold a data type, then pointed at
+ * a value in the ServerConfig class. When the value has been
+ * read and validated, the Set method is called to write the
+ * value safely in a type-safe manner.
  */
-bool InitClasses(ServerConfig* conf, const char* tag);
+template<typename T> class ValueContainer : public ValueContainerBase
+{
+	T ServerConfig::* const vptr;
+ public:
+	/** Initialize with a value of type T */
+	ValueContainer(T ServerConfig::* const offset) : vptr(offset)
+	{
+	}
+
+	/** Change value to type T of size s */
+	void Set(ServerConfig* conf, const T& value)
+	{
+		conf->*vptr = value;
+	}
+
+	void Set(ServerConfig* conf, const ValueItem& item);
+};
+
+template<> void ValueContainer<char[MAXBUF]>::Set(ServerConfig* conf, ValueItem const& item);
+
+
+class ValueContainerLimit : public ValueContainerBase
+{
+	size_t ServerLimits::* const vptr;
+ public:
+	/** Initialize with a value of type T */
+	ValueContainerLimit(size_t ServerLimits::* const offset) : vptr(offset)
+	{
+	}
+
+	/** Change value to type T of size s */
+	void Set(ServerConfig* conf, const size_t& value)
+	{
+		conf->Limits.*vptr = value;
+	}
+};
 
-/** Initialize an oper type
+/** A specialization of ValueContainer to hold a pointer to a bool
  */
-bool DoType(ServerConfig* conf, const char* tag, char** entries, ValueList &values, int* types);
+typedef ValueContainer<bool> ValueContainerBool;
 
-/** Initialize an oper class
+/** A specialization of ValueContainer to hold a pointer to
+ * an unsigned int
  */
-bool DoClass(ServerConfig* conf, const char* tag, char** entries, ValueList &values, int* types);
+typedef ValueContainer<unsigned int> ValueContainerUInt;
 
-/** Finish initializing the oper types and classes
+/** A specialization of ValueContainer to hold a pointer to
+ * a char array.
  */
-bool DoneClassesAndTypes(ServerConfig* conf, const char* tag);
+typedef ValueContainer<char[MAXBUF]> ValueContainerChar;
 
+/** A specialization of ValueContainer to hold a pointer to
+ * a char array.
+ */
+typedef ValueContainer<std::string> ValueContainerString;
 
+/** A specialization of ValueContainer to hold a pointer to
+ * an int
+ */
+typedef ValueContainer<int> ValueContainerInt;
 
-/** Initialize x line
+/** A set of ValueItems used by multi-value validator functions
  */
-bool InitXLine(ServerConfig* conf, const char* tag);
+typedef std::deque<ValueItem> ValueList;
 
-/** Add a config-defined zline
+/** A callback for validating a single value
  */
-bool DoZLine(ServerConfig* conf, const char* tag, char** entries, ValueList &values, int* types);
+typedef bool (*Validator)(ServerConfig* conf, const char*, const char*, ValueItem&);
+/** A callback for validating multiple value entries
+ */
+typedef bool (*MultiValidator)(ServerConfig* conf, const char*, const char**, ValueList&, int*);
+/** A callback indicating the end of a group of entries
+ */
+typedef bool (*MultiNotify)(ServerConfig* conf, const char*);
 
-/** Add a config-defined qline
+/** Holds a core configuration item and its callbacks
  */
-bool DoQLine(ServerConfig* conf, const char* tag, char** entries, ValueList &values, int* types);
+struct InitialConfig
+{
+	/** Tag name */
+	const char* tag;
+	/** Value name */
+	const char* value;
+	/** Default, if not defined */
+	const char* default_value;
+	/** Value containers */
+	ValueContainerBase* val;
+	/** Data types */
+	int datatype;
+	/** Validation function */
+	Validator validation_function;
+};
 
-/** Add a config-defined kline
+/** Represents a deprecated configuration tag.
  */
-bool DoKLine(ServerConfig* conf, const char* tag, char** entries, ValueList &values, int* types);
+struct Deprecated
+{
+	/** Tag name
+	 */
+	const char* tag;
+	/** Tag value
+	 */
+	const char* value;
+	/** Reason for deprecation
+	 */
+	const char* reason;
+};
 
-/** Add a config-defined eline
+/** Holds a core configuration item and its callbacks
+ * where there may be more than one item
  */
-bool DoELine(ServerConfig* conf, const char* tag, char** entries, ValueList &values, int* types);
+struct MultiConfig
+{
+	/** Tag name */
+	const char*	tag;
+	/** One or more items within tag */
+	const char*	items[MAX_VALUES_PER_TAG];
+	/** One or more defaults for items within tags */
+	const char* items_default[MAX_VALUES_PER_TAG];
+	/** One or more data types */
+	int		datatype[MAX_VALUES_PER_TAG];
+	/** Validation function */
+	MultiValidator	validation_function;
+};
 
 #endif
diff --git a/include/inspircd.h b/include/inspircd.h
index ef6c86f28..2609f3cc1 100644
--- a/include/inspircd.h
+++ b/include/inspircd.h
@@ -611,11 +611,6 @@ class CoreExport InspIRCd : public classbase
 	 */
 	Channel* FindChan(const char* chan);
 
-	/** Check for a 'die' tag in the config file, and abort if found
-	 * @return Depending on the configuration, this function may never return
-	 */
-	void CheckDie();
-
 	/** Check we aren't running as root, and exit if we are
 	 * @return Depending on the configuration, this function may never return
 	 */
diff --git a/include/users.h b/include/users.h
index 9d1072f57..175ad0d0f 100644
--- a/include/users.h
+++ b/include/users.h
@@ -74,9 +74,8 @@ class UserResolver;
 
 /** Holds information relevent to &lt;connect allow&gt; and &lt;connect deny&gt; tags in the config file.
  */
-class CoreExport ConnectClass : public classbase
+struct CoreExport ConnectClass : public classbase
 {
- private:
 	/** Type of line, either CC_ALLOW or CC_DENY
 	 */
 	char type;
@@ -129,141 +128,33 @@ class CoreExport ConnectClass : public classbase
 	 */
 	int port;
 
-public:
-
-	/** Create a new connect class based on an existing connect class. This is required for std::vector (at least under windows).
-	 */
-	ConnectClass(const ConnectClass* source) : classbase(), type(source->type), name(source->name),
-		registration_timeout(source->registration_timeout), host(source->host),
-		pingtime(source->pingtime), pass(source->pass), hash(source->hash), sendqmax(source->sendqmax),
-		recvqmax(source->recvqmax), maxlocal(source->maxlocal), maxglobal(source->maxglobal), maxchans(source->maxchans),
-		port(source->port), RefCount(0), disabled(false), limit(source->limit)
-	{
-	}
-
-	/** Create a new connect class with no settings.
-	 */
-	ConnectClass() : type(CC_DENY), name("unnamed"), registration_timeout(0), host(""), pingtime(0), pass(""), hash(""),
-			sendqmax(0), recvqmax(0), maxlocal(0), maxglobal(0), RefCount(0), disabled(false), limit(0)
-	{
-	}
-
-	/** Create a new connect class to ALLOW connections.
-	 * @param thename Name of the connect class
-	 * @param timeout The registration timeout
-	 * @param hst The IP mask to allow
-	 * @param ping The ping frequency
-	 * @param pas The password to be used
-	 * @param hsh The hash to be used
-	 * @param sendq The maximum sendq value
-	 * @param recvq The maximum recvq value
-	 * @param maxl The maximum local sessions
-	 * @param maxg The maximum global sessions
-	 */
-	ConnectClass(const std::string &thename, unsigned int timeout,const std::string &hst, unsigned int ping,
-			const std::string &pas, const std::string &hsh, unsigned long sendq, unsigned long recvq,
-			unsigned long maxl, unsigned long maxg, unsigned int maxc, int p = 0) :
-			type(CC_ALLOW), name(thename), registration_timeout(timeout), host(hst), pingtime(ping), pass(pas), hash(hsh),
-			sendqmax(sendq), recvqmax(recvq), maxlocal(maxl), maxglobal(maxg), maxchans(maxc), port(p), RefCount(0), disabled(false), limit(0) { }
-
-	/** Create a new connect class to DENY connections
-	 * @param thename Name of the connect class
-	 * @param hst The IP mask to deny
-	 */
-	ConnectClass(const std::string &thename, const std::string &hst) : type(CC_DENY), name(thename), registration_timeout(0),
-			host(hst), pingtime(0), pass(""), hash(""), sendqmax(0), recvqmax(0), maxlocal(0), maxglobal(0), maxchans(0), port(0), RefCount(0), disabled(false), limit(0)
-	{
-	}
-
-	/* Create a new connect class based on another class
-	 * @param thename The name of the connect class
-	 * @param source Another connect class to inherit all but the name from
+	/** How many users may be in this connect class before they are refused?
+	 * (0 = no limit = default)
 	 */
-	ConnectClass(const std::string &thename, const ConnectClass* source) : type(source->type), name(thename),
-				registration_timeout(source->registration_timeout), host(source->host),
-				pingtime(source->pingtime), pass(source->pass), hash(source->hash), sendqmax(source->sendqmax),
-				recvqmax(source->recvqmax), maxlocal(source->maxlocal), maxglobal(source->maxglobal), maxchans(source->maxchans),
-				port(source->port), RefCount(0), disabled(false), limit(source->limit)
-	{
-	}
-
-	void SetDisabled(bool t)
-	{
-		this->disabled = t;
-	}
-
-	bool GetDisabled()
-	{
-		return this->disabled;
-	}
-
-	/* Update an existing entry with new values
-	 */
-	void Update(unsigned int timeout, const std::string &hst, unsigned int ping,
-				const std::string &pas, unsigned long sendq, unsigned long recvq,
-				unsigned long maxl, unsigned long maxg, unsigned int maxc, int p, unsigned long llimit)
-	{
-		if (timeout)
-			registration_timeout = timeout;
-		if (!hst.empty())
-			host = hst;
-		if (ping)
-			pingtime = ping;
-		if (!pas.empty())
-			pass = pas;
-		if (sendq)
-			sendqmax = sendq;
-		if (recvq)
-			recvqmax = recvq;
-		if (maxl)
-			maxlocal = maxl;
-		if (maxg)
-			maxglobal = maxg;
-		if (maxc)
-			maxchans = maxc;
-		if (p)
-			port = p;
-
-		this->limit = llimit;
-	}
-
-	void Update(const std::string &n, const std::string &hst)
-	{
-		name = n;
-		host = hst;
-	}
+	unsigned long limit;
 
-	/** Reference counter. Contains an int as to how many users are connected to this class. :)
-	 * This will be 0 if no users are connected. If a <connect> is removed from the config, and there
-	 * are 0 users on it - it will go away in RAM. :)
+	/** Reference counter.
+	 * This will be 1 if no users are connected, as long as it is a valid connect block
+	 * When it reaches 0, the object should be deleted
 	 */
 	unsigned long RefCount;
 
-	/** If this is true, any attempt to set a user to this class will fail. Default false. This is really private, it's only in the public section thanks to the way this class is written
-	 */
-	bool disabled;
-
-	/** How many users may be in this connect class before they are refused? (0 = disabled = default)
+	/** Create a new connect class with no settings.
 	 */
-	unsigned long limit;
-
-	size_t GetMaxChans()
-	{
-		return maxchans;
-	}
-
-	/** Returns the type, CC_ALLOW or CC_DENY
+	ConnectClass(char type, const std::string& mask);
+	/** Create a new connect class with inherited settings.
 	 */
-	char GetType()
-	{
-		return (type == CC_ALLOW ? CC_ALLOW : CC_DENY);
-	}
+	ConnectClass(char type, const std::string& mask, const ConnectClass& parent);
+	
+	/** Update the settings in this block to match the given block */
+	void Update(const ConnectClass* newSettings);
 
-	std::string& GetName()
-	{
-		return name;
-	}
 
+	const std::string& GetName() { return name; }
+	const std::string& GetPass() { return pass; }
+	const std::string& GetHost() { return host; }
+	const int GetPort() { return port; }
+	
 	/** Returns the registration timeout
 	 */
 	time_t GetRegTimeout()
@@ -271,27 +162,6 @@ public:
 		return (registration_timeout ? registration_timeout : 90);
 	}
 
-	/** Returns the allowed or denied IP mask
-	 */
-	const std::string& GetHost()
-	{
-		return host;
-	}
-
-	/** Get port number
-	 */
-	int GetPort()
-	{
-		return port;
-	}
-
-	/** Set port number
-	 */
-	void SetPort(int p)
-	{
-		port = p;
-	}
-
 	/** Returns the ping frequency
 	 */
 	unsigned int GetPingTime()
@@ -299,20 +169,6 @@ public:
 		return (pingtime ? pingtime : 120);
 	}
 
-	/** Returns the password or an empty string
-	 */
-	const std::string& GetPass()
-	{
-		return pass;
-	}
-
-	/** Returns the hash or an empty string
-	 */
-	const std::string& GetHash()
-	{
-		return hash;
-	}
-
 	/** Returns the maximum sendq value
 	 */
 	unsigned long GetSendqMax()