11235654657

Committed 08 Oct 2024 12:41PM UTC coverage: 64.665% (-0.03%) from 64.694%

Build # 11235654657

Build Type

push

github

Committed by

web-flow

Commit Message

Merge pull request #14754 from omoerbeek/rec-unittest-cache-init

rec: make sure the record cache has sane paramers for each unit test

Run Details

37099 of 88084 branches covered (42.12%)

Branch coverage included in aggregate %.

4 of 4 new or added lines in 3 files covered. (100.0%)

88 existing lines in 14 files now uncovered.

124704 of 162134 relevant lines covered (76.91%)

4249786.58 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

67.07

/pdns/axfr-retriever.cc

/*
 * This file is part of PowerDNS or dnsdist.
 * Copyright -- PowerDNS.COM B.V. and its contributors
 *
 * This program is free software; you can redistribute it and/or modify
 * it under the terms of version 2 of the GNU General Public License as
 * published by the Free Software Foundation.
 *
 * In addition, for the avoidance of any doubt, permission is granted to
 * link this program with OpenSSL and to (re)distribute the binaries
 * produced as the result of such linking.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU General Public License for more details.
 *
 * You should have received a copy of the GNU General Public License
 * along with this program; if not, write to the Free Software
 * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
 */

#include "axfr-retriever.hh"
#include "arguments.hh"
#include "dns_random.hh"
#include "utility.hh"
#include "resolver.hh"
#include "query-local-address.hh"

using pdns::resolver::parseResult;

AXFRRetriever::AXFRRetriever(const ComboAddress& remote,
                             const DNSName& domain,
                             const TSIGTriplet& tt, 
                             const ComboAddress* laddr,
                             size_t maxReceivedBytes,
                             uint16_t timeout)
  : d_buf(65536), d_tsigVerifier(tt, remote, d_trc), d_receivedBytes(0), d_maxReceivedBytes(maxReceivedBytes)
{
  ComboAddress local;
  if (laddr != nullptr) {
    local = ComboAddress(*laddr);
  } else {
    if (!pdns::isQueryLocalAddressFamilyEnabled(remote.sin4.sin_family)) {
      throw ResolverException("Unable to determine source address for AXFR request to " + remote.toStringWithPort() + " for " + domain.toLogString() + ". Address family is not configured for outgoing queries");
    }
    local = pdns::getQueryLocalAddress(remote.sin4.sin_family, 0);
  }
  d_sock = -1;
  try {
    d_sock = makeQuerySocket(local, false); // make a TCP socket
    if (d_sock < 0)
      throw ResolverException("Error creating socket for AXFR request to "+d_remote.toStringWithPort());

    d_remote = remote; // mostly for error reporting
    this->connect(timeout);
    d_soacount = 0;
  
    vector<uint8_t> packet;
    DNSPacketWriter pw(packet, domain, QType::AXFR);
    pw.getHeader()->id = dns_random_uint16();
  
    if(!tt.name.empty()) {
      if (tt.algo == DNSName("hmac-md5"))
        d_trc.d_algoName = tt.algo + DNSName("sig-alg.reg.int");
      else
        d_trc.d_algoName = tt.algo;
      d_trc.d_time = time(nullptr);
      d_trc.d_fudge = 300;
      d_trc.d_origID=ntohs(pw.getHeader()->id);
      d_trc.d_eRcode=0;
      addTSIG(pw, d_trc, tt.name, tt.secret, "", false);
    }
  
    uint16_t replen=htons(packet.size());
    Utility::iovec iov[2];
    iov[0].iov_base=reinterpret_cast<char*>(&replen);
    iov[0].iov_len=2;
    iov[1].iov_base=packet.data();
    iov[1].iov_len=packet.size();
  
    int ret=Utility::writev(d_sock, iov, 2);
    if(ret < 0)
      throw ResolverException("Error sending question to "+d_remote.toStringWithPort()+": "+stringerror());
    if(ret != (int)(2+packet.size())) {
      throw ResolverException("Partial write on AXFR request to "+d_remote.toStringWithPort());
    }
  
    int res = waitForData(d_sock, timeout, 0);
    
    if(!res)
      throw ResolverException("Timeout waiting for answer from "+d_remote.toStringWithPort()+" during AXFR");
    if(res<0)
      throw ResolverException("Error waiting for answer from "+d_remote.toStringWithPort()+": "+stringerror());
  }
  catch(...) {
    if(d_sock >= 0)
      close(d_sock);
    d_sock = -1;
    throw;
  }
}

AXFRRetriever::~AXFRRetriever()
{
  close(d_sock);
}



int AXFRRetriever::getChunk(Resolver::res_t &res, vector<DNSRecord>* records, uint16_t timeout) // Implementation is making sure RFC2845 4.4 is followed.
{
  if(d_soacount > 1)
    return false;

  // d_sock is connected and is about to spit out a packet
  int len=getLength(timeout);
  if(len<0)
    throw ResolverException("EOF trying to read axfr chunk from remote TCP client");

  if (d_maxReceivedBytes > 0 && (d_maxReceivedBytes - d_receivedBytes) < (size_t) len)
    throw ResolverException("Reached the maximum number of received bytes during AXFR");

  timeoutReadn(len, timeout);

  d_receivedBytes += (uint16_t) len;

  MOADNSParser mdp(false, d_buf.data(), len);

  int err = mdp.d_header.rcode;

  if(err) {
    throw ResolverException("AXFR chunk error: " + RCode::to_s(err));
  }

  if(mdp.d_header.tc) {
    throw ResolverException("AXFR chunk had TC bit set");
  }

  try {
    d_tsigVerifier.check(std::string(d_buf.data(), len), mdp);
  }
  catch(const std::runtime_error& re) {
    throw ResolverException(re.what());
  }

  if(!records) {
    err = parseResult(mdp, DNSName(), 0, 0, &res);

    if (!err) {
      for(const auto& answer :  mdp.d_answers) {
        if (answer.d_type == QType::SOA) {
          d_soacount++;
        }
      }
    }
  }
  else {
    records->clear();
    records->reserve(mdp.d_answers.size());

    for(auto& r: mdp.d_answers) {
      if (r.d_type == QType::SOA) {
        d_soacount++;
      }

      records->push_back(std::move(r));
    }
  }

  return true;
}

void AXFRRetriever::timeoutReadn(uint16_t bytes, uint16_t timeoutsec)
{
  const time_t start = time(nullptr);
  uint16_t bytesRead = 0;

  while (bytesRead < bytes) {
    // coverity[store_truncates_time_t]
    auto elapsed = time(nullptr) - start;
    if (elapsed > timeoutsec) {
      throw ResolverException("Timeout while reading data from remote nameserver over TCP");
    }
    auto res = waitForData(d_sock, static_cast<int>(timeoutsec - elapsed));
    if (res < 0) {
      throw ResolverException("Reading data from remote nameserver over TCP: " + stringerror());
    }
    if (res == 0) {
      throw ResolverException("Timeout while reading data from remote nameserver over TCP");
    }

    auto received = recv(d_sock, &d_buf.at(bytesRead), bytes - bytesRead, 0);
    if (received < 0) {
      throw ResolverException("Reading data from remote nameserver over TCP: " + stringerror());
    }
    if (received == 0) {
      throw ResolverException("Remote nameserver closed TCP connection");
    }
    bytesRead += static_cast<uint16_t>(received);
  }
}

void AXFRRetriever::connect(uint16_t timeout)
{
  setNonBlocking( d_sock );

  int err;

  if((err=::connect(d_sock,(struct sockaddr*)&d_remote, d_remote.getSocklen()))<0 && errno!=EINPROGRESS) {
    try {
      closesocket(d_sock);
    }
    catch(const PDNSException& e) {
      d_sock=-1;
      throw ResolverException("Error closing AXFR socket after connect() failed: "+e.reason);
    }

    throw ResolverException("connect: "+stringerror());
  }

  if(!err)
    goto done;

  err=waitForRWData(d_sock, false, timeout, 0); // wait for writeability
  
  if(!err) {
    try {
      closesocket(d_sock); // timeout
    }
    catch(const PDNSException& e) {
      d_sock=-1;
      throw ResolverException("Error closing AXFR socket after timeout: "+e.reason);
    }

    d_sock=-1;
    errno=ETIMEDOUT;
    
    throw ResolverException("Timeout connecting to server");
  }
  else if(err < 0) {
    throw ResolverException("Error connecting: "+stringerror());
  }
  else {
    Utility::socklen_t len=sizeof(err);
    if(getsockopt(d_sock, SOL_SOCKET,SO_ERROR,(char *)&err,&len)<0)
      throw ResolverException("Error connecting: "+stringerror()); // Solaris

    if(err)
      throw ResolverException("Error connecting: "+string(strerror(err)));
  }
  
 done:
  setBlocking( d_sock );
  // d_sock now connected
}

int AXFRRetriever::getLength(uint16_t timeout)
{
  timeoutReadn(2, timeout);
  return (unsigned char)d_buf.at(0)*256+(unsigned char)d_buf.at(1);
}


1	/*
2	* This file is part of PowerDNS or dnsdist.
3	* Copyright -- PowerDNS.COM B.V. and its contributors
4	*
5	* This program is free software; you can redistribute it and/or modify
6	* it under the terms of version 2 of the GNU General Public License as
7	* published by the Free Software Foundation.
8	*
9	* In addition, for the avoidance of any doubt, permission is granted to
10	* link this program with OpenSSL and to (re)distribute the binaries
11	* produced as the result of such linking.
12	*
13	* This program is distributed in the hope that it will be useful,
14	* but WITHOUT ANY WARRANTY; without even the implied warranty of
15	* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
16	* GNU General Public License for more details.
17	*
18	* You should have received a copy of the GNU General Public License
19	* along with this program; if not, write to the Free Software
20	* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
21	*/
22
23	#include "axfr-retriever.hh"
24	#include "arguments.hh"
25	#include "dns_random.hh"
26	#include "utility.hh"
27	#include "resolver.hh"
28	#include "query-local-address.hh"
29
30	using pdns::resolver::parseResult;
31
32	AXFRRetriever::AXFRRetriever(const ComboAddress& remote,
33	const DNSName& domain,
34	const TSIGTriplet& tt,
35	const ComboAddress* laddr,
36	size_t maxReceivedBytes,
37	uint16_t timeout)
38	: d_buf(65536), d_tsigVerifier(tt, remote, d_trc), d_receivedBytes(0), d_maxReceivedBytes(maxReceivedBytes)
39	{	460✔
40	ComboAddress local;	460✔
41	if (laddr != nullptr) {	460!
42	local = ComboAddress(*laddr);	460✔
43	} else {	460✔
UNCOV 44	if (!pdns::isQueryLocalAddressFamilyEnabled(remote.sin4.sin_family)) {	×
45	throw ResolverException("Unable to determine source address for AXFR request to " + remote.toStringWithPort() + " for " + domain.toLogString() + ". Address family is not configured for outgoing queries");	×
46	}	×
UNCOV 47	local = pdns::getQueryLocalAddress(remote.sin4.sin_family, 0);	×
UNCOV 48	}	×
49	d_sock = -1;	460✔
50	try {	460✔
51	d_sock = makeQuerySocket(local, false); // make a TCP socket	460✔
52	if (d_sock < 0)	460!
53	throw ResolverException("Error creating socket for AXFR request to "+d_remote.toStringWithPort());	×
54
55	d_remote = remote; // mostly for error reporting	460✔
56	this->connect(timeout);	460✔
57	d_soacount = 0;	460✔
58
59	vector<uint8_t> packet;	460✔
60	DNSPacketWriter pw(packet, domain, QType::AXFR);	460✔
61	pw.getHeader()->id = dns_random_uint16();	460✔
62
63	if(!tt.name.empty()) {	460✔
64	if (tt.algo == DNSName("hmac-md5"))	23!
65	d_trc.d_algoName = tt.algo + DNSName("sig-alg.reg.int");	23✔
66	else	×
67	d_trc.d_algoName = tt.algo;	×
68	d_trc.d_time = time(nullptr);	23✔
69	d_trc.d_fudge = 300;	23✔
70	d_trc.d_origID=ntohs(pw.getHeader()->id);	23✔
71	d_trc.d_eRcode=0;	23✔
72	addTSIG(pw, d_trc, tt.name, tt.secret, "", false);	23✔
73	}	23✔
74
75	uint16_t replen=htons(packet.size());	460✔
76	Utility::iovec iov[2];	460✔
77	iov[0].iov_base=reinterpret_cast<char*>(&replen);	460✔
78	iov[0].iov_len=2;	460✔
79	iov[1].iov_base=packet.data();	460✔
80	iov[1].iov_len=packet.size();	460✔
81
82	int ret=Utility::writev(d_sock, iov, 2);	460✔
83	if(ret < 0)	460!
84	throw ResolverException("Error sending question to "+d_remote.toStringWithPort()+": "+stringerror());	×
85	if(ret != (int)(2+packet.size())) {	460!
86	throw ResolverException("Partial write on AXFR request to "+d_remote.toStringWithPort());	×
87	}	×
88
89	int res = waitForData(d_sock, timeout, 0);	460✔
90
91	if(!res)	460!
92	throw ResolverException("Timeout waiting for answer from "+d_remote.toStringWithPort()+" during AXFR");	×
93	if(res<0)	460!
94	throw ResolverException("Error waiting for answer from "+d_remote.toStringWithPort()+": "+stringerror());	×
95	}	460✔
96	catch(...) {	460✔
97	if(d_sock >= 0)	2!
98	close(d_sock);	2✔
99	d_sock = -1;	2✔
100	throw;	2✔
101	}	2✔
102	}	460✔
103
104	AXFRRetriever::~AXFRRetriever()
105	{	458✔
106	close(d_sock);	458✔
107	}	458✔
108
109
110
111	int AXFRRetriever::getChunk(Resolver::res_t &res, vector<DNSRecord>* records, uint16_t timeout) // Implementation is making sure RFC2845 4.4 is followed.
112	{	19,980✔
113	if(d_soacount > 1)	19,980✔
114	return false;	455✔
115
116	// d_sock is connected and is about to spit out a packet
117	int len=getLength(timeout);	19,525✔
118	if(len<0)	19,525!
119	throw ResolverException("EOF trying to read axfr chunk from remote TCP client");	×
120
121	if (d_maxReceivedBytes > 0 && (d_maxReceivedBytes - d_receivedBytes) < (size_t) len)	19,525!
122	throw ResolverException("Reached the maximum number of received bytes during AXFR");	×
123
124	timeoutReadn(len, timeout);	19,525✔
125
126	d_receivedBytes += (uint16_t) len;	19,525✔
127
128	MOADNSParser mdp(false, d_buf.data(), len);	19,525✔
129
130	int err = mdp.d_header.rcode;	19,525✔
131
132	if(err) {	19,525!
133	throw ResolverException("AXFR chunk error: " + RCode::to_s(err));	×
134	}	×
135
136	if(mdp.d_header.tc) {	19,525!
137	throw ResolverException("AXFR chunk had TC bit set");	×
138	}	×
139
140	try {	19,525✔
141	d_tsigVerifier.check(std::string(d_buf.data(), len), mdp);	19,525✔
142	}	19,525✔
143	catch(const std::runtime_error& re) {	19,525✔
144	throw ResolverException(re.what());	×
145	}	×
146
147	if(!records) {	19,525✔
148	err = parseResult(mdp, DNSName(), 0, 0, &res);	19,436✔
149
150	if (!err) {	19,436!
151	for(const auto& answer : mdp.d_answers) {	1,501,500✔
152	if (answer.d_type == QType::SOA) {	1,501,500✔
153	d_soacount++;	896✔
154	}	896✔
155	}	1,501,500✔
156	}	19,436✔
157	}	19,436✔
158	else {	89✔
159	records->clear();	89✔
160	records->reserve(mdp.d_answers.size());	89✔
161
162	for(auto& r: mdp.d_answers) {	25,072✔
163	if (r.d_type == QType::SOA) {	25,072✔
164	d_soacount++;	20✔
165	}	20✔
166
167	records->push_back(std::move(r));	25,072✔
168	}	25,072✔
169	}	89✔
170
171	return true;	19,525✔
172	}	19,525✔
173
174	void AXFRRetriever::timeoutReadn(uint16_t bytes, uint16_t timeoutsec)
175	{	39,050✔
176	const time_t start = time(nullptr);	39,050✔
177	uint16_t bytesRead = 0;	39,050✔
178
179	while (bytesRead < bytes) {	78,114✔
180	// coverity[store_truncates_time_t]
181	auto elapsed = time(nullptr) - start;	39,064✔
182	if (elapsed > timeoutsec) {	39,064!
183	throw ResolverException("Timeout while reading data from remote nameserver over TCP");	×
184	}	×
185	auto res = waitForData(d_sock, static_cast<int>(timeoutsec - elapsed));	39,064✔
186	if (res < 0) {	39,064!
187	throw ResolverException("Reading data from remote nameserver over TCP: " + stringerror());	×
188	}	×
189	if (res == 0) {	39,064!
190	throw ResolverException("Timeout while reading data from remote nameserver over TCP");	×
191	}	×
192
193	auto received = recv(d_sock, &d_buf.at(bytesRead), bytes - bytesRead, 0);	39,064✔
194	if (received < 0) {	39,064!
195	throw ResolverException("Reading data from remote nameserver over TCP: " + stringerror());	×
196	}	×
197	if (received == 0) {	39,064!
198	throw ResolverException("Remote nameserver closed TCP connection");	×
199	}	×
200	bytesRead += static_cast<uint16_t>(received);	39,064✔
201	}	39,064✔
202	}	39,050✔
203
204	void AXFRRetriever::connect(uint16_t timeout)
205	{	460✔
206	setNonBlocking( d_sock );	460✔
207
208	int err;	460✔
209
210	if((err=::connect(d_sock,(struct sockaddr*)&d_remote, d_remote.getSocklen()))<0 && errno!=EINPROGRESS) {	460!
211	try {	×
212	closesocket(d_sock);	×
213	}	×
214	catch(const PDNSException& e) {	×
215	d_sock=-1;	×
216	throw ResolverException("Error closing AXFR socket after connect() failed: "+e.reason);	×
217	}	×
218
219	throw ResolverException("connect: "+stringerror());	×
220	}	×
221
222	if(!err)	460!
223	goto done;	×
224
225	err=waitForRWData(d_sock, false, timeout, 0); // wait for writeability	460✔
226
227	if(!err) {	460!
228	try {	×
229	closesocket(d_sock); // timeout	×
230	}	×
231	catch(const PDNSException& e) {	×
232	d_sock=-1;	×
233	throw ResolverException("Error closing AXFR socket after timeout: "+e.reason);	×
234	}	×
235
236	d_sock=-1;	×
237	errno=ETIMEDOUT;	×
238
239	throw ResolverException("Timeout connecting to server");	×
240	}	×
241	else if(err < 0) {	460!
242	throw ResolverException("Error connecting: "+stringerror());	×
243	}	×
244	else {	460✔
245	Utility::socklen_t len=sizeof(err);	460✔
246	if(getsockopt(d_sock, SOL_SOCKET,SO_ERROR,(char *)&err,&len)<0)	460!
247	throw ResolverException("Error connecting: "+stringerror()); // Solaris	×
248
249	if(err)	460✔
250	throw ResolverException("Error connecting: "+string(strerror(err)));	2✔
251	}	460✔
252
253	done:	458✔
254	setBlocking( d_sock );	458✔
255	// d_sock now connected
256	}	458✔
257
258	int AXFRRetriever::getLength(uint16_t timeout)
259	{	19,525✔
260	timeoutReadn(2, timeout);	19,525✔
261	return (unsigned char)d_buf.at(0)*256+(unsigned char)d_buf.at(1);	19,525✔
262	}	19,525✔
263

PowerDNS / pdns / 11235654657

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous