Explicitly call the std::string version of WriteServ in chanrec::UserList to prevent...

[user/henk/code/inspircd.git] / src / channels.cpp

diff --git a/src/channels.cpp b/src/channels.cpp
index f340e12bf1528b4edf9a75cc3ddb8aeff0e72129..732b253cc7d4208cae31c1e6bcae1c7f4a9ef087 100644 (file)
--- 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;
                                                }
                                        }
                                }
@@ -425,6 +418,7 @@ chanrec* chanrec::ForceChan(InspIRCd* Instance, chanrec* Ptr,ucrec *a,userrec* u
 
        a->channel = Ptr;
        Ptr->AddUser(user);
+       user->ModChannelCount(1);
        Ptr->WriteChannel(user,"JOIN :%s",Ptr->name);
 
        /* Major improvement by Brain - we dont need to be calculating all this pointlessly for remote users */
@@ -441,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
@@ -467,6 +480,7 @@ long chanrec::PartUser(userrec *user, const char* reason)
                        }
                        user->chans[i]->uc_modes = 0;
                        user->chans[i]->channel = NULL;
+                       user->ModChannelCount(-1);
                        this->RemoveAllPrefixes(user);
                        break;
                }
@@ -838,7 +852,7 @@ void chanrec::UserList(userrec *user)
                if (curlen > (480-NICKMAX))
                {
                        /* list overflowed into multiple numerics */
-                       user->WriteServ(list);
+                       user->WriteServ(std::string(list));
 
                        /* reset our lengths */
                        dlen = curlen = snprintf(list,MAXBUF,"353 %s = %s :", user->nick, this->name);
@@ -852,7 +866,7 @@ void chanrec::UserList(userrec *user)
        /* if whats left in the list isnt empty, send it */
        if (numusers)
        {
-               user->WriteServ(list);
+               user->WriteServ(std::string(list));
        }
 
        user->WriteServ("366 %s %s :End of /NAMES list.", user->nick, this->name);