X-Git-Url: https://git.netwichtig.de/gitweb/?a=blobdiff_plain;f=src%2Fmessage.cpp;h=268cc65540617635467ffc534503a82fa837ae0a;hb=922d4ebf7a27a6577d6b4f91e0f42ccdfa989455;hp=91744dd50d3835d27070ec30fc4273bbbab172a3;hpb=af284a476e55d7cc9af23a23a08153cda07d6570;p=user%2Fhenk%2Fcode%2Finspircd.git diff --git a/src/message.cpp b/src/message.cpp index 91744dd50..268cc6554 100644 --- a/src/message.cpp +++ b/src/message.cpp @@ -69,15 +69,14 @@ int common_channels(userrec *u, userrec *u2) for (std::vector::const_iterator i = u->chans.begin(); i != u->chans.end(); i++) { /* Fetch the channel from the user */ - ucrec* user_channel = (ucrec*)(*i); + ucrec* user_channel = *i; if (user_channel->channel) { /* Eliminate the inner loop (which used to be ~equal in size to the outer loop) * by replacing it with a map::find which *should* be more efficient */ - CUList* channel_user_map = user_channel->channel->GetUsers(); - if (channel_user_map->find(u2) != channel_user_map->end()) + if (user_channel->channel->HasUser(u2)) return 1; } } @@ -96,33 +95,11 @@ void NonBlocking(int s) fcntl(s, F_SETFL, flags | O_NONBLOCK); } -int CleanAndResolve (char *resolvedHost, const char *unresolvedHost, bool forward) -{ - bool ok; - std::string ipaddr; - - DNS d(Config->DNSServer); - if (forward) - ok = d.ForwardLookup(unresolvedHost, false); - else - ok = d.ReverseLookup(unresolvedHost, false); - if (!ok) - return 0; - time_t T = time(NULL)+1; - while ((!d.HasResult()) && (time(NULL)::const_iterator i = u->chans.begin(); i != u->chans.end(); i++) - if (((ucrec*)(*i))->channel) + if ((*i)->channel) z++; return z; @@ -169,7 +146,7 @@ int isident(const char* n) { continue; } - if (strchr(".-0123456789",*i)) + if (((*i >= '0') && (*i <= '9')) || (*i == '-') || (*i == '.')) { continue; } @@ -188,13 +165,13 @@ int isnick(const char* n) int p = 0; for (char* i = (char*)n; *i; i++, p++) { - /* can occur anywhere in a nickname */ + /* "A"-"}" can occur anywhere in a nickname */ if ((*i >= 'A') && (*i <= '}')) { continue; } - /* can occur anywhere BUT the first char of a nickname */ - if ((strchr("-0123456789",*i)) && (i > n)) + /* "0"-"9", "-" can occur anywhere BUT the first char of a nickname */ + if ((((*i >= '0') && (*i <= '9')) || (*i == '-')) && (i > n)) { continue; } @@ -218,17 +195,17 @@ const char* cmode(userrec *user, chanrec *chan) for (std::vector::const_iterator i = user->chans.begin(); i != user->chans.end(); i++) { - if (((ucrec*)(*i))->channel == chan) + if ((*i)->channel == chan) { - if ((((ucrec*)(*i))->uc_modes & UCMODE_OP) > 0) + if (((*i)->uc_modes & UCMODE_OP) > 0) { return "@"; } - if ((((ucrec*)(*i))->uc_modes & UCMODE_HOP) > 0) + if (((*i)->uc_modes & UCMODE_HOP) > 0) { return "%"; } - if ((((ucrec*)(*i))->uc_modes & UCMODE_VOICE) > 0) + if (((*i)->uc_modes & UCMODE_VOICE) > 0) { return "+"; } @@ -245,9 +222,9 @@ int cflags(userrec *user, chanrec *chan) for (std::vector::const_iterator i = user->chans.begin(); i != user->chans.end(); i++) { - if (((ucrec*)(*i))->channel == chan) + if ((*i)->channel == chan) { - return ((ucrec*)(*i))->uc_modes; + return (*i)->uc_modes; } } return 0; @@ -270,17 +247,17 @@ int cstatus(userrec *user, chanrec *chan) for (std::vector::const_iterator i = user->chans.begin(); i != user->chans.end(); i++) { - if (((ucrec*)(*i))->channel == chan) + if ((*i)->channel == chan) { - if ((((ucrec*)(*i))->uc_modes & UCMODE_OP) > 0) + if (((*i)->uc_modes & UCMODE_OP) > 0) { return STATUS_OP; } - if ((((ucrec*)(*i))->uc_modes & UCMODE_HOP) > 0) + if (((*i)->uc_modes & UCMODE_HOP) > 0) { return STATUS_HOP; } - if ((((ucrec*)(*i))->uc_modes & UCMODE_VOICE) > 0) + if (((*i)->uc_modes & UCMODE_VOICE) > 0) { return STATUS_VOICE; } @@ -290,34 +267,6 @@ int cstatus(userrec *user, chanrec *chan) return STATUS_NORMAL; } -void TidyBan(char *ban) -{ - if (!ban) { - log(DEFAULT,"*** BUG *** TidyBan was given an invalid parameter"); - return; - } - - char temp[MAXBUF],NICK[MAXBUF],IDENT[MAXBUF],HOST[MAXBUF]; - - strlcpy(temp,ban,MAXBUF); - - char* pos_of_pling = strchr(temp,'!'); - char* pos_of_at = strchr(temp,'@'); - - pos_of_pling[0] = '\0'; - pos_of_at[0] = '\0'; - pos_of_pling++; - pos_of_at++; - - strlcpy(NICK,temp,NICKMAX-1); - strlcpy(IDENT,pos_of_pling,IDENTMAX+1); - strlcpy(HOST,pos_of_at,63); - - snprintf(ban,MAXBUF,"%s!%s@%s",NICK,IDENT,HOST); -} - -char lst[MAXBUF]; - std::string chlist(userrec *user,userrec* source) { std::string list; @@ -330,24 +279,16 @@ std::string chlist(userrec *user,userrec* source) ucrec* rec = *i; if(rec->channel && rec->channel->name) - { - /* XXX - Why does this check need to be here at all? :< */ - /* Commenting this out until someone finds a case where we need it */ - //if (lst.find(rec->channel->name) == std::string::npos) - //{ - - /* - * If the target is the same as the sender, let them see all their channels. - * If the channel is NOT private/secret AND the user is not invisible. - * If the user is an oper, and the option is set. - */ - if ((source == user) || (*source->oper && Config->OperSpyWhois) || (((!rec->channel->modes[CM_PRIVATE]) && (!rec->channel->modes[CM_SECRET]) && !(user->modes[UM_INVISIBLE])) || (rec->channel->HasUser(source)))) - { - list.append(cmode(user, rec->channel)).append(rec->channel->name).append(" "); - } - //} + { + /* If the target is the same as the sender, let them see all their channels. + * If the channel is NOT private/secret OR the user shares a common channel + * If the user is an oper, and the option is set. + */ + if ((source == user) || (*source->oper && Config->OperSpyWhois) || (((!rec->channel->modes[CM_PRIVATE]) && (!rec->channel->modes[CM_SECRET])) || (rec->channel->HasUser(source)))) + { + list.append(cmode(user, rec->channel)).append(rec->channel->name).append(" "); + } } } - return list; }