From 8edd5bb4526c356b350d21146f0d23ff9d61a90d Mon Sep 17 00:00:00 2001 From: Remi Gacogne Date: Wed, 28 Jun 2017 10:57:49 +0200 Subject: [PATCH] rec: Initialize MemRecursorCache::d_state in the ctor It's always set in `MemRecursorCache::replace()`, which should be the only place where we insert new values, but the explicit init makes Coverity happy. --- pdns/recursor_cache.hh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pdns/recursor_cache.hh b/pdns/recursor_cache.hh index 831a86f61..e928d91c2 100644 --- a/pdns/recursor_cache.hh +++ b/pdns/recursor_cache.hh @@ -74,7 +74,7 @@ private: struct CacheEntry { CacheEntry(const boost::tuple& key, const vector>& records, bool auth) : - d_records(records), d_qname(key.get<0>()), d_netmask(key.get<2>()), d_ttd(0), d_qtype(key.get<1>()), d_auth(auth) + d_records(records), d_qname(key.get<0>()), d_netmask(key.get<2>()), d_state(Indeterminate), d_ttd(0), d_qtype(key.get<1>()), d_auth(auth) {} typedef vector> records_t; @@ -84,7 +84,7 @@ private: } records_t d_records; - vector> d_signatures; + std::vector> d_signatures; std::vector> d_authorityRecs; DNSName d_qname; Netmask d_netmask; -- 2.40.0