diff options
author | Casey Deccio <casey@deccio.net> | 2021-02-06 01:59:44 +0300 |
---|---|---|
committer | Casey Deccio <casey@deccio.net> | 2021-02-06 01:59:44 +0300 |
commit | ad7422ad1b023374fb94aeb41cbe019233b416de (patch) | |
tree | f7df51f2429910099c8f5154674aab54ffacbabd | |
parent | 46321ba65434a67ed1d258c6d886ff57ec66207e (diff) |
Don't try to retrieve a query that wasn't made
-rw-r--r-- | dnsviz/analysis/offline.py | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/dnsviz/analysis/offline.py b/dnsviz/analysis/offline.py index 6b7b6b2..afc257b 100644 --- a/dnsviz/analysis/offline.py +++ b/dnsviz/analysis/offline.py @@ -256,6 +256,13 @@ class OfflineDomainNameAnalysis(OnlineDomainNameAnalysis): return info_obj def _get_response_info(self, name, rdtype): + #XXX there are reasons for this (e.g., NXDOMAIN, after which no further + # queries are made), but it would be good to have a sanity check, so + # we don't simply produce an incomplete output. + # see also: dnsviz.viz.dnssec.graph_rrset_auth() + if (name, rdtype) not in self.queries: + return None + query = self.queries[(name, rdtype)] name_to_info_mapping = {} rrset_to_cname_mapping = {} |