From 0ed6f92ebe94d679206b91a174688bca21eab4be Mon Sep 17 00:00:00 2001 From: Sadie Powell Date: Sun, 12 Apr 2020 20:56:43 +0100 Subject: Switch the dnsbl module from reference<> to std::shared_ptr<>. --- src/modules/m_dnsbl.cpp | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) (limited to 'src/modules/m_dnsbl.cpp') diff --git a/src/modules/m_dnsbl.cpp b/src/modules/m_dnsbl.cpp index 3eee1f347..61cc1a255 100644 --- a/src/modules/m_dnsbl.cpp +++ b/src/modules/m_dnsbl.cpp @@ -32,7 +32,7 @@ #include "modules/stats.h" /* Class holding data for a single entry */ -class DNSBLConfEntry : public refcountbase +class DNSBLConfEntry { public: enum EnumBanaction { I_UNKNOWN, I_KILL, I_ZLINE, I_KLINE, I_GLINE, I_MARK }; @@ -57,10 +57,10 @@ class DNSBLResolver : public DNS::Request std::string theiruid; StringExtItem& nameExt; IntExtItem& countExt; - reference ConfEntry; + std::shared_ptr ConfEntry; public: - DNSBLResolver(DNS::Manager *mgr, Module *me, StringExtItem& match, IntExtItem& ctr, const std::string &hostname, LocalUser* u, reference conf) + DNSBLResolver(DNS::Manager *mgr, Module *me, StringExtItem& match, IntExtItem& ctr, const std::string &hostname, LocalUser* u, std::shared_ptr conf) : DNS::Request(mgr, me, hostname, DNS::QUERY_A, true) , theirsa(u->client_sa) , theiruid(u->uuid) @@ -239,7 +239,7 @@ class DNSBLResolver : public DNS::Request } }; -typedef std::vector > DNSBLConfList; +typedef std::vector> DNSBLConfList; class ModuleDNSBL : public Module, public Stats::EventListener { @@ -289,7 +289,7 @@ class ModuleDNSBL : public Module, public Stats::EventListener for(ConfigIter i = dnsbls.first; i != dnsbls.second; ++i) { ConfigTag* tag = i->second; - reference e = new DNSBLConfEntry(); + auto e = std::make_shared(); e->name = tag->getString("name"); e->ident = tag->getString("ident"); @@ -451,13 +451,13 @@ class ModuleDNSBL : public Module, public Stats::EventListener unsigned long total_hits = 0, total_misses = 0; - for (std::vector >::const_iterator i = DNSBLConfEntries.begin(); i != DNSBLConfEntries.end(); ++i) + for (std::shared_ptr e : DNSBLConfEntries) { - total_hits += (*i)->stats_hits; - total_misses += (*i)->stats_misses; + total_hits += e->stats_hits; + total_misses += e->stats_misses; - stats.AddRow(304, "DNSBLSTATS DNSbl \"" + (*i)->name + "\" had " + - ConvToStr((*i)->stats_hits) + " hits and " + ConvToStr((*i)->stats_misses) + " misses"); + stats.AddRow(304, "DNSBLSTATS DNSbl \"" + e->name + "\" had " + + ConvToStr(e->stats_hits) + " hits and " + ConvToStr(e->stats_misses) + " misses"); } stats.AddRow(304, "DNSBLSTATS Total hits: " + ConvToStr(total_hits)); -- cgit v1.3.1-10-gc9f91