From 29fd51d6f76a639fbd909e7bf7489eb92e9f90db Mon Sep 17 00:00:00 2001
From: brain
Date: Sat, 2 Sep 2006 14:09:45 +0000
Subject: Add chanrec::IsBanned() so that we dont have to keep walking the
 banlist in various modules and using match()

git-svn-id: http://svn.inspircd.org/repository/trunk/inspircd@5099 e03df62e-2008-0410-955e-edbf42e46eb7
---
 src/channels.cpp | 32 ++++++++++++++++++++++----------
 1 file changed, 22 insertions(+), 10 deletions(-)

(limited to 'src/channels.cpp')

diff --git a/src/channels.cpp b/src/channels.cpp
index 5c3bb266f..dab518844 100644
--- a/src/channels.cpp
+++ b/src/channels.cpp
@@ -318,17 +318,10 @@ chanrec* chanrec::JoinUser(InspIRCd* Instance, userrec *user, const char* cn, bo
 					sprintf(mask,"%s!%s@%s",user->nick, user->ident, user->GetIPString());
 					if (!MOD_RESULT)
 					{
-						for (BanList::iterator i = Ptr->bans.begin(); i != Ptr->bans.end(); i++)
+						if (Ptr->IsBanned(user))
 						{
-							/* This allows CIDR ban matching
-							 * 
-							 *	  Full masked host			Full unmasked host		     IP with/without CIDR
-							 */
-							if ((match(user->GetFullHost(),i->data)) || (match(user->GetFullRealHost(),i->data)) || (match(mask, i->data, true)))
-							{
-								user->WriteServ("474 %s %s :Cannot join channel (You're banned)",user->nick, Ptr->name);
-								return NULL;
-							}
+							user->WriteServ("474 %s %s :Cannot join channel (You're banned)",user->nick, Ptr->name);
+							return NULL;
 						}
 					}
 				}
@@ -442,6 +435,25 @@ chanrec* chanrec::ForceChan(InspIRCd* Instance, chanrec* Ptr,ucrec *a,userrec* u
 	return Ptr;
 }
 
+bool chanrec::IsBanned(userrec* user)
+{
+	char mask[MAXBUF];
+	sprintf(mask,"%s!%s@%s",user->nick, user->ident, user->GetIPString());
+	for (BanList::iterator i = this->bans.begin(); i != this->bans.end(); i++)
+	{
+		/* This allows CIDR ban matching
+		 * 
+		 *        Full masked host                      Full unmasked host                   IP with/without CIDR
+		 */
+		if ((match(user->GetFullHost(),i->data)) || (match(user->GetFullRealHost(),i->data)) || (match(mask, i->data, true)))
+		{
+			return true;
+		}
+
+	}
+	return false;
+}
+
 /* chanrec::PartUser
  * remove a channel from a users record, and remove the record from the hash
  * if the channel has become empty
-- 
cgit v1.3.1-10-gc9f91