#include "inspircd_config.h"
#include "inspircd.h"
-#include "inspircd_io.h"
-#include <unistd.h>
-#include <fcntl.h>
-#include <sys/errno.h>
-#include <time.h>
#include <string>
-#ifdef GCC3
-#include <ext/hash_map>
-#else
-#include <hash_map>
-#endif
#include <map>
#include <sstream>
#include <vector>
#include "commands.h"
#include "xline.h"
#include "inspstring.h"
+#include "inspircd.h"
#include "helperfuncs.h"
#include "hashcomp.h"
#include "typedefs.h"
#include "cull_list.h"
-CullItem::CullItem(userrec* u, std::string r)
+extern InspIRCd* ServerInstance;
+extern user_hash clientlist;
+
+/*
+ * In current implementation of CullList, this isn't used. It did odd things with a lot of sockets.
+ */
+bool CullList::IsValid(userrec* user)
+{
+ time_t esignon = 0;
+ std::map<userrec*,time_t>::iterator es = exempt.find(user);
+ if (es != exempt.end())
+ esignon = es->second;
+
+ for (user_hash::iterator u = clientlist.begin(); u != clientlist.end(); u++)
+ {
+ /*
+ * BUGFIX
+ *
+ * Because there is an undetermined period of time between a user existing,
+ * and this function being called, we have to check for the following condition:
+ *
+ * Between CullList::AddItem(u) being called, and CullList::IsValid(u) being called,
+ * the user with the pointer u has quit, but only to be REPLACED WITH A NEW USER WHO
+ * BECAUSE OF ALLOCATION RULES, HAS THE SAME MEMORY ADDRESS! To prevent this, we
+ * cross reference each pointer to the user's signon time, and if the signon times
+ * do not match, we return false here to indicate this user is NOT valid as it
+ * seems to differ from the pointer snapshot we got a few seconds earlier. Should
+ * prevent a few random crashes during netsplits.
+ */
+ if (user == u->second)
+ return (u->second->signon == esignon);
+ }
+ return false;
+}
+
+CullItem::CullItem(userrec* u, std::string &r)
+{
+ this->user = u;
+ this->reason = r;
+}
+
+CullItem::CullItem(userrec* u, const char* r)
+{
+ this->user = u;
+ this->reason = r;
+}
+
+CullItem::~CullItem()
{
- this->user = u;
- this->reason = r;
}
userrec* CullItem::GetUser()
{
- return this->user;
+ return this->user;
}
-std::string CullItem::GetReason()
+std::string& CullItem::GetReason()
{
- return this->reason;
+ return this->reason;
}
CullList::CullList()
{
list.clear();
- exempt.clear();
+ exempt.clear();
+}
+
+void CullList::AddItem(userrec* user, std::string &reason)
+{
+ if (exempt.find(user) == exempt.end())
+ {
+ CullItem item(user,reason);
+ list.push_back(item);
+ exempt[user] = user->signon;
+ }
}
-void CullList::AddItem(userrec* user, std::string reason)
+void CullList::AddItem(userrec* user, const char* reason)
{
if (exempt.find(user) == exempt.end())
{
- CullItem item(user,reason);
- list.push_back(item);
- exempt[user] = 1;
+ CullItem item(user,reason);
+ list.push_back(item);
+ exempt[user] = user->signon;
}
}
int CullList::Apply()
{
- int n = 0;
- while (list.size())
- {
- std::vector<CullItem>::iterator a = list.begin();
- userrec* u = a->GetUser();
- std::string reason = a->GetReason();
- kill_link(u,reason.c_str());
- list.erase(list.begin());
- n++;
- }
- return n;
+ int n = list.size();
+ while (list.size())
+ {
+ std::vector<CullItem>::iterator a = list.begin();
+
+ kill_link(a->GetUser(), a->GetReason().c_str());
+ list.erase(list.begin());
+ }
+ return n;
}