bug 1501283 - for NS checks we don't have originSuffix r=valentin
authorDaniel Stenberg <daniel@haxx.se>
Tue, 23 Oct 2018 20:22:54 +0000
changeset 491012 94f5958dbd0ff027a4b4fa65bcc2c956ba8c0d47
parent 491011 b872e2aa271c063a31219ee93455541198b99cbc
child 491013 3daa70ac76f606894a99371cae7312d8a93a2de5
push id247
push userfmarier@mozilla.com
push dateSat, 27 Oct 2018 01:06:44 +0000
reviewersvalentin
bugs1501283, 1500549
milestone65.0a1
bug 1501283 - for NS checks we don't have originSuffix r=valentin This fixes the regression caused by bug 1500549. MozReview-Commit-ID: 3VBvIbrEbDT Differential Revision: https://phabricator.services.mozilla.com/D9526
netwerk/dns/TRRService.cpp
--- a/netwerk/dns/TRRService.cpp
+++ b/netwerk/dns/TRRService.cpp
@@ -671,17 +671,17 @@ TRRService::CompleteLookup(nsHostRecord 
     return LOOKUP_OK;
   }
 
   // when called without a host record, this is a domain name check response.
   if (NS_SUCCEEDED(status)) {
     LOG(("TRR verified %s to be fine!\n", newRRSet->mHostName.get()));
   } else {
     LOG(("TRR says %s doesn't resolve as NS!\n", newRRSet->mHostName.get()));
-    TRRBlacklist(newRRSet->mHostName, rec->originSuffix, pb, false);
+    TRRBlacklist(newRRSet->mHostName, nsCString(""), pb, false);
   }
   return LOOKUP_OK;
 }
 
 AHostResolver::LookupStatus
 TRRService::CompleteLookupByType(nsHostRecord *, nsresult,
                                  const nsTArray<nsCString> *aResult,
                                  uint32_t aTtl,