core_hostname_lookup: find answer record of the correct type instead of assuming...

[user/henk/code/inspircd.git] / src / modules / m_flashpolicyd.cpp

diff --git a/src/modules/m_flashpolicyd.cpp b/src/modules/m_flashpolicyd.cpp
index 46fd33e612e818b91915e01ac907de3407ddc596..8f847e1117772feb24050241344fdb2876e8913a 100644 (file)
--- a/src/modules/m_flashpolicyd.cpp
+++ b/src/modules/m_flashpolicyd.cpp
@@ -23,20 +23,30 @@ class FlashPDSocket;
 
 namespace
 {
-       std::set<FlashPDSocket*> sockets;
+       insp::intrusive_list<FlashPDSocket> sockets;
        std::string policy_reply;
        const std::string expected_request("<policy-file-request/>\0", 23);
 }
 
-class FlashPDSocket : public BufferedSocket
+class FlashPDSocket : public BufferedSocket, public Timer, public insp::intrusive_list_node<FlashPDSocket>
 {
- public:
-       time_t created;
+       /** True if this object is in the cull list
+        */
+       bool waitingcull;
+
+       bool Tick(time_t currtime) CXX11_OVERRIDE
+       {
+               AddToCull();
+               return false;
+       }
 
-       FlashPDSocket(int newfd)
+ public:
+       FlashPDSocket(int newfd, unsigned int timeoutsec)
                : BufferedSocket(newfd)
-               , created(ServerInstance->Time())
+               , Timer(timeoutsec)
+               , waitingcull(false)
        {
+               ServerInstance->Timers.AddTimer(this);
        }
 
        ~FlashPDSocket()
@@ -58,10 +68,10 @@ class FlashPDSocket : public BufferedSocket
 
        void AddToCull()
        {
-               if (created == 0)
+               if (waitingcull)
                        return;
 
-               created = 0;
+               waitingcull = true;
                Close();
                ServerInstance->GlobalCulls.AddItem(this);
        }
@@ -69,19 +79,9 @@ class FlashPDSocket : public BufferedSocket
 
 class ModuleFlashPD : public Module
 {
-       time_t timeout;
+       unsigned int timeout;
 
  public:
-       void OnBackgroundTimer(time_t curtime) CXX11_OVERRIDE
-       {
-               for (std::set<FlashPDSocket*>::const_iterator i = sockets.begin(); i != sockets.end(); ++i)
-               {
-                       FlashPDSocket* sock = *i;
-                       if ((sock->created + timeout <= curtime) && (sock->created != 0))
-                               sock->AddToCull();
-               }
-       }
-
        ModResult OnAcceptConnection(int nfd, ListenSocket* from, irc::sockets::sockaddrs* client, irc::sockets::sockaddrs* server) CXX11_OVERRIDE
        {
                if (from->bind_tag->getString("type") != "flashpolicyd")
@@ -90,7 +90,7 @@ class ModuleFlashPD : public Module
                if (policy_reply.empty())
                        return MOD_RES_DENY;
 
-               sockets.insert(new FlashPDSocket(nfd));
+               sockets.push_front(new FlashPDSocket(nfd, timeout));
                return MOD_RES_ALLOW;
        }
 
@@ -148,7 +148,7 @@ class ModuleFlashPD : public Module
 
        CullResult cull()
        {
-               for (std::set<FlashPDSocket*>::const_iterator i = sockets.begin(); i != sockets.end(); ++i)
+               for (insp::intrusive_list<FlashPDSocket>::const_iterator i = sockets.begin(); i != sockets.end(); ++i)
                {
                        FlashPDSocket* sock = *i;
                        sock->AddToCull();