From 93fe642d4396949740824bddc6ca11968f2f53e9 Mon Sep 17 00:00:00 2001 From: Pieter Lexis Date: Thu, 28 Jul 2016 14:21:10 +0200 Subject: [PATCH] Recursor: Fix a possible crash When the parser creates empty DNSNames (for whatever reason) and `rec_control current-queries` is run, the process would abort because it tried to print an empty DNSName. --- pdns/rec_channel_rec.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pdns/rec_channel_rec.cc b/pdns/rec_channel_rec.cc index a209e822a6..336f431251 100644 --- a/pdns/rec_channel_rec.cc +++ b/pdns/rec_channel_rec.cc @@ -593,7 +593,7 @@ static string* pleaseGetCurrentQueries() for(MT_t::waiters_t::iterator mthread=MT->d_waiters.begin(); mthread!=MT->d_waiters.end() && n < 100; ++mthread, ++n) { const PacketID& pident = mthread->key; ostr << (fmt - % pident.domain.toString() /* ?? */ % DNSRecordContent::NumberToType(pident.type) + % pident.domain.toLogString() /* ?? */ % DNSRecordContent::NumberToType(pident.type) % pident.remote.toString() % (pident.sock ? 'Y' : 'n') % (pident.fd == -1 ? 'Y' : 'n') ); -- 2.47.2