X-Git-Url: https://git.netwichtig.de/gitweb/?a=blobdiff_plain;f=src%2Fuserprocess.cpp;h=42c3bd3db49aff8bd84fc971eb4a9a19e978b42c;hb=eb28eaea35d9d109a0b7b890de9d957d562da675;hp=2adccc36e5f419d1e1515f1f8b8b3aa08314c76c;hpb=d54fbc71e2a2e0f70002e2d36c669c4aa3d59e17;p=user%2Fhenk%2Fcode%2Finspircd.git diff --git a/src/userprocess.cpp b/src/userprocess.cpp index 2adccc36e..42c3bd3db 100644 --- a/src/userprocess.cpp +++ b/src/userprocess.cpp @@ -28,19 +28,11 @@ using namespace std; #include #include #include -#ifdef GCC3 #include -#else -#include -#endif #include #include #include #include -#include -#ifdef THREADED_DNS -#include -#endif #include "users.h" #include "ctables.h" #include "globals.h" @@ -52,7 +44,6 @@ using namespace std; #include "commands.h" #include "xline.h" #include "inspstring.h" -#include "dnsqueue.h" #include "helperfuncs.h" #include "hashcomp.h" #include "socketengine.h" @@ -65,21 +56,11 @@ extern struct sockaddr_in client,server; extern socklen_t length; extern std::vector modules; extern std::vector factory; -extern std::vector module_sockets; extern time_t TIME; extern time_t OLDTIME; -extern std::vector local_users; -extern InspSocket* socket_ref[MAX_DESCRIPTORS]; - -extern InspIRCd* ServerInstance; -extern ServerConfig *Config; -extern userrec* fd_ref_table[MAX_DESCRIPTORS]; char data[65536]; -extern user_hash clientlist; -extern chan_hash chanlist; - -void ProcessUser(userrec* cu) +void InspIRCd::ProcessUser(userrec* cu) { int result = EAGAIN; @@ -88,14 +69,14 @@ void ProcessUser(userrec* cu) log(DEBUG,"Processing user with fd %d",cu->fd); - if (Config->GetIOHook(cu->port)) + if (this->Config->GetIOHook(cu->GetPort())) { int result2 = 0; int MOD_RESULT = 0; try { - MOD_RESULT = Config->GetIOHook(cu->port)->OnRawSocketRead(cu->fd,data,65535,result2); + MOD_RESULT = this->Config->GetIOHook(cu->GetPort())->OnRawSocketRead(cu->fd,data,65535,result2); log(DEBUG,"Data result returned by module: %d",MOD_RESULT); } catch (ModuleException& modexcept) @@ -123,30 +104,19 @@ void ProcessUser(userrec* cu) { userrec *current; int currfd; - int floodlines; + int floodlines = 0; - ServerInstance->stats->statsRecv += result; + this->stats->statsRecv += result; /* * perform a check on the raw buffer as an array (not a string!) to remove - * characters 0 and 7 which are illegal in the RFC - replace them with spaces. - * hopefully this should stop even more people whining about "Unknown command: *" - */ - - /* - * XXX - potential replacement for the below using my beloved pointers. --w00t + * character 0 which is illegal in the RFC - replace them with spaces. * XXX - no garauntee there's not \0's in the middle of the data, * and no reason for it to be terminated either. -- Om - * - * for (char *c = data; data && *data; data++) - * { - * if (*data == 0 || *data == 7) - * data = ' '; - * } */ for (int checker = 0; checker < result; checker++) { - if ((data[checker] == 0) || (data[checker] == 7)) + if (data[checker] == 0) data[checker] = ' '; } @@ -155,7 +125,6 @@ void ProcessUser(userrec* cu) current = cu; currfd = current->fd; - floodlines = 0; // add the data to the users buffer if (result > 0) @@ -163,7 +132,7 @@ void ProcessUser(userrec* cu) if (!current->AddBuffer(data)) { // AddBuffer returned false, theres too much data in the user's buffer and theyre up to no good. - if (current->registered == 7) + if (current->registered == REG_ALL) { // Make sure they arn't flooding long lines. if (TIME > current->reset_due) @@ -178,37 +147,37 @@ void ProcessUser(userrec* cu) { log(DEFAULT,"Excess flood from: %s!%s@%s",current->nick,current->ident,current->host); WriteOpers("*** Excess flood from: %s!%s@%s",current->nick,current->ident,current->host); - kill_link(current,"Excess flood"); + userrec::QuitUser(this, current,"Excess flood"); return; } else { - WriteServ(currfd, "NOTICE %s :Your previous line was too long and was not delivered (Over 512chars) Please shorten it.", current->nick); + current->WriteServ("NOTICE %s :Your previous line was too long and was not delivered (Over 512chars) Please shorten it.", current->nick); current->recvq = ""; } } else { - WriteOpers("*** Excess flood from %s",(char*)inet_ntoa(current->ip4)); - log(DEFAULT,"Excess flood from: %s",(char*)inet_ntoa(current->ip4)); - add_zline(120,Config->ServerName,"Flood from unregistered connection",(char*)inet_ntoa(current->ip4)); + WriteOpers("*** Excess flood from %s",current->GetIPString()); + log(DEFAULT,"Excess flood from: %s",current->GetIPString()); + add_zline(120,this->Config->ServerName,"Flood from unregistered connection",current->GetIPString()); apply_lines(APPLY_ZLINES); } return; } - if (current->recvq.length() > (unsigned)Config->NetBufferSize) + if (current->recvq.length() > (unsigned)this->Config->NetBufferSize) { - if (current->registered == 7) + if (current->registered == REG_ALL) { - kill_link(current,"RecvQ exceeded"); + userrec::QuitUser(this, current,"RecvQ exceeded"); } else { - WriteOpers("*** Excess flood from %s",(char*)inet_ntoa(current->ip4)); - log(DEFAULT,"Excess flood from: %s",(char*)inet_ntoa(current->ip4)); - add_zline(120,Config->ServerName,"Flood from unregistered connection",(char*)inet_ntoa(current->ip4)); + WriteOpers("*** Excess flood from %s",current->GetIPString()); + log(DEFAULT,"Excess flood from: %s",current->GetIPString()); + add_zline(120,this->Config->ServerName,"Flood from unregistered connection",current->GetIPString()); apply_lines(APPLY_ZLINES); } @@ -218,36 +187,31 @@ void ProcessUser(userrec* cu) // while there are complete lines to process... while (current->BufferIsReady()) { - char sanitized[MAXBUF]; - - floodlines++; if (TIME > current->reset_due) { current->reset_due = TIME + current->threshold; current->lines_in = 0; } - current->lines_in++; - - if (current->lines_in > current->flood) + if (++current->lines_in > current->flood) { log(DEFAULT,"Excess flood from: %s!%s@%s",current->nick,current->ident,current->host); WriteOpers("*** Excess flood from: %s!%s@%s",current->nick,current->ident,current->host); - kill_link(current,"Excess flood"); + userrec::QuitUser(this, current,"Excess flood"); return; } - if ((floodlines > current->flood) && (current->flood != 0)) + if ((++floodlines > current->flood) && (current->flood != 0)) { - if (current->registered == 7) + if (current->registered == REG_ALL) { log(DEFAULT,"Excess flood from: %s!%s@%s",current->nick,current->ident,current->host); WriteOpers("*** Excess flood from: %s!%s@%s",current->nick,current->ident,current->host); - kill_link(current,"Excess flood"); + userrec::QuitUser(this, current,"Excess flood"); } else { - add_zline(120,Config->ServerName,"Flood from unregistered connection",(char*)inet_ntoa(current->ip4)); + add_zline(120,this->Config->ServerName,"Flood from unregistered connection",current->GetIPString()); apply_lines(APPLY_ZLINES); } @@ -258,29 +222,27 @@ void ProcessUser(userrec* cu) std::string single_line = current->GetBuffer(); current->bytes_in += single_line.length(); current->cmds_in++; - strlcpy(sanitized,single_line.c_str(),511); - - if (*sanitized) + if (single_line.length() > 512) + single_line.resize(512); + + userrec* old_comp = this->fd_ref_table[currfd]; + + this->Parser->ProcessBuffer(single_line,current); + /* + * look for the user's record in case it's changed... if theyve quit, + * we cant do anything more with their buffer, so bail. + * there used to be an ugly, slow loop here. Now we have a reference + * table, life is much easier (and FASTER) + */ + userrec* new_comp = this->fd_ref_table[currfd]; + if ((currfd < 0) || (!this->fd_ref_table[currfd]) || (old_comp != new_comp)) { - userrec* old_comp = fd_ref_table[currfd]; - - ServerInstance->Parser->ProcessBuffer(sanitized,current); - /* - * look for the user's record in case it's changed... if theyve quit, - * we cant do anything more with their buffer, so bail. - * there used to be an ugly, slow loop here. Now we have a reference - * table, life is much easier (and FASTER) - */ - userrec* new_comp = fd_ref_table[currfd]; - if ((currfd < 0) || (!fd_ref_table[currfd]) || (old_comp != new_comp)) - { - return; - } - else - { - /* The user is still here, flush their buffer */ - current->FlushWriteBuf(); - } + return; + } + else + { + /* The user is still here, flush their buffer */ + current->FlushWriteBuf(); } } @@ -290,7 +252,7 @@ void ProcessUser(userrec* cu) if ((result == -1) && (errno != EAGAIN) && (errno != EINTR)) { log(DEBUG,"killing: %s",cu->nick); - kill_link(cu,strerror(errno)); + userrec::QuitUser(this,cu,strerror(errno)); return; } } @@ -303,32 +265,32 @@ void ProcessUser(userrec* cu) else if (result == 0) { log(DEBUG,"InspIRCd: Exited: %s",cu->nick); - kill_link(cu,"Client exited"); + userrec::QuitUser(this,cu,"Client exited"); log(DEBUG,"Bailing from client exit"); return; } } -void DoSocketTimeouts(time_t TIME) +void InspIRCd::DoSocketTimeouts(time_t TIME) { - unsigned int numsockets = module_sockets.size(); - SocketEngine* SE = ServerInstance->SE; + unsigned int numsockets = this->module_sockets.size(); + SocketEngine* SE = this->SE; - for (std::vector::iterator a = module_sockets.begin(); a < module_sockets.end(); a++) + for (std::vector::iterator a = this->module_sockets.begin(); a < this->module_sockets.end(); a++) { InspSocket* s = (InspSocket*)*a; - if ((s) && (s->GetFd() >= 0) && (s->GetFd() < MAX_DESCRIPTORS) && (socket_ref[s->GetFd()] != NULL) && (s->Timeout(TIME))) + if ((s) && (s->GetFd() >= 0) && (s->GetFd() < MAX_DESCRIPTORS) && (this->socket_ref[s->GetFd()] != NULL) && (s->Timeout(TIME))) { log(DEBUG,"userprocess.cpp: Socket poll returned false, close and bail"); - socket_ref[s->GetFd()] = NULL; + this->socket_ref[s->GetFd()] = NULL; SE->DelFd(s->GetFd()); - module_sockets.erase(a); + this->module_sockets.erase(a); s->Close(); - delete s; + DELETE(s); break; } - if (module_sockets.size() != numsockets) + if (this->module_sockets.size() != numsockets) break; } } @@ -339,19 +301,17 @@ void DoSocketTimeouts(time_t TIME) * stuff like ping checks, registration timeouts, etc. This function is * also responsible for checking if InspSocket derived classes are timed out. */ -void DoBackgroundUserStuff(time_t TIME) +void InspIRCd::DoBackgroundUserStuff(time_t TIME) { CullList GlobalGoners; - for (std::vector::iterator count2 = local_users.begin(); count2 != local_users.end(); count2++) + /* XXX: IT IS NOT SAFE TO USE AN ITERATOR HERE. DON'T EVEN THINK ABOUT IT. */ + for (unsigned long count2 = 0; count2 != this->local_users.size(); count2++) { - /* Sanity checks for corrupted iterators (yes, really) */ - userrec* curr = NULL; + if (count2 >= this->local_users.size()) + break; - if (*count2) - curr = (userrec*)(*count2); - if ((long)curr == -1) - return; + userrec* curr = this->local_users[count2]; if (curr) { @@ -359,43 +319,44 @@ void DoBackgroundUserStuff(time_t TIME) * registration timeout -- didnt send USER/NICK/HOST * in the time specified in their connection class. */ - if (((unsigned)TIME > (unsigned)curr->timeout) && (curr->registered != 7)) + if (((unsigned)TIME > (unsigned)curr->timeout) && (curr->registered != REG_ALL)) { log(DEBUG,"InspIRCd: registration timeout: %s",curr->nick); + //ZapThisDns(curr->fd); GlobalGoners.AddItem(curr,"Registration timeout"); continue; } - /* * user has signed on with USER/NICK/PASS, and dns has completed, all the modules * say this user is ok to proceed, fully connect them. */ - if ((TIME > curr->signon) && (curr->registered == 3) && (AllModulesReportReady(curr))) + if ((TIME > curr->signon) && (curr->registered == REG_NICKUSER) && (AllModulesReportReady(curr))) { curr->dns_done = true; - ServerInstance->stats->statsDnsBad++; - FullConnectUser(curr,&GlobalGoners); + //ZapThisDns(curr->fd); + this->stats->statsDnsBad++; + curr->FullConnect(&GlobalGoners); continue; } - - if ((curr->dns_done) && (curr->registered == 3) && (AllModulesReportReady(curr))) + if ((curr->dns_done) && (curr->registered == REG_NICKUSER) && (AllModulesReportReady(curr))) { log(DEBUG,"dns done, registered=3, and modules ready, OK"); - FullConnectUser(curr,&GlobalGoners); + curr->FullConnect(&GlobalGoners); + //ZapThisDns(curr->fd); continue; } - // It's time to PING this user. Send them a ping. - if ((TIME > curr->nping) && (curr->registered == 7)) + if ((TIME > curr->nping) && (curr->registered == REG_ALL)) { // This user didn't answer the last ping, remove them if (!curr->lastping) { GlobalGoners.AddItem(curr,"Ping timeout"); + curr->lastping = 1; + curr->nping = TIME+curr->pingmax; continue; } - - Write(curr->fd,"PING :%s",Config->ServerName); + curr->Write("PING :%s",this->Config->ServerName); curr->lastping = 0; curr->nping = TIME+curr->pingmax; } @@ -404,16 +365,18 @@ void DoBackgroundUserStuff(time_t TIME) * We can flush the write buffer as the last thing we do, because if they * match any of the above conditions its no use flushing their buffer anyway. */ + curr->FlushWriteBuf(); - if (*curr->GetWriteError()) { GlobalGoners.AddItem(curr,curr->GetWriteError()); continue; } } + } + /* Remove all the queued users who are due to be quit, free memory used. */ GlobalGoners.Apply(); }