* Returns 1 if colliding local client, 2 if colliding remote, 3 if colliding both.
* Sends SVSNICKs as appropriate and forces nickchanges too.
*/
-int TreeSocket::DoCollision(User *u, time_t remotets, const char *remoteident, const char *remoteip, const char *remoteuid)
+int TreeSocket::DoCollision(User *u, time_t remotets, const std::string &remoteident, const std::string &remoteip, const std::string &remoteuid)
{
/*
* Under old protocol rules, we would have had to kill both clients.
* These days, we have UID. And, so what we do is, force nick change client(s)
* involved according to timestamp rules.
*
- * RULES:
- * user@ip equal:
- * Force nick change on OLDER timestamped client
- * user@ip differ:
- * Force nick change on NEWER timestamped client
- * TS EQUAL:
- * FNC both.
- *
- * This stops abusive use of collisions, simplifies problems with loops, and so on.
+ * RULES:
+ * user@ip equal:
+ * Force nick change on OLDER timestamped client
+ * user@ip differ:
+ * Force nick change on NEWER timestamped client
+ * TS EQUAL:
+ * FNC both.
+ *
+ * This stops abusive use of collisions, simplifies problems with loops, and so on.
* -- w00t
*/
bool bChangeLocal = true;
bool bChangeRemote = true;
- /* for brevity, don't use the User */
- time_t localts = u->age;
- const char *localident = u->ident.c_str();
- const char *localip = u->GetIPString();
+ /* for brevity, don't use the User - use defines to avoid any copy */
+ #define localts u->age
+ #define localident u->ident
+ #define localip u->GetIPString()
/* mmk. let's do this again. */
if (remotets == localts)
/* fuck. now it gets complex. */
/* first, let's see if ident@host matches. */
- bool SamePerson = !strcmp(localident, remoteident)
- && !strcmp(localip, remoteip);
+ bool SamePerson = (localident == remoteident)
+ && (localip == remoteip);
/*
* if ident@ip is equal, and theirs is newer, or
* have 928AAAB's nick set to that.
* -- w00t
*/
- User *remote = this->Instance->FindUUID(remoteuid);
+ User *remote = this->ServerInstance->FindUUID(remoteuid);
if (remote)
{
else
{
/* user has not been introduced yet, just inform their server */
- this->WriteLine(std::string(":")+this->Instance->Config->GetSID()+" SVSNICK "+remoteuid+" " + remoteuid + " " + ConvToStr(remotets));
+ this->WriteLine(std::string(":")+this->ServerInstance->Config->GetSID()+" SVSNICK "+remoteuid+" " + remoteuid + " " + ConvToStr(remotets));
}
if (!bChangeLocal)