dnsforward -- fix panic on ANY request

dnsforward/querylog_top.go

@@ -168,7 +168,7 @@ func (r *dayTop) addEntry(entry *logEntry, q *dns.Msg, now time.Time) error {
 		return err
 	}
 
-	if entry.Result.IsFiltered {
+	if entry.Result != nil && entry.Result.IsFiltered {
 		err := runningTop.hours[hour].incrementBlocked(hostname)
 		if err != nil {
 			log.Printf("Failed to increment value: %s", err)

dnsforward/stats.go

@@ -193,24 +193,27 @@ func (h *histogram) Observe(value float64) {
 // -----
 func incrementCounters(entry *logEntry) {
 	requests.IncWithTime(entry.Time)
-	if entry.Result.IsFiltered {
-		filtered.IncWithTime(entry.Time)
-	}
-	switch entry.Result.Reason {
-	case dnsfilter.NotFilteredWhiteList:
-		whitelisted.IncWithTime(entry.Time)
-	case dnsfilter.NotFilteredError:
-		errorsTotal.IncWithTime(entry.Time)
-	case dnsfilter.FilteredBlackList:
-		filteredLists.IncWithTime(entry.Time)
-	case dnsfilter.FilteredSafeBrowsing:
-		filteredSafebrowsing.IncWithTime(entry.Time)
-	case dnsfilter.FilteredParental:
-		filteredParental.IncWithTime(entry.Time)
-	case dnsfilter.FilteredInvalid:
-		// do nothing
-	case dnsfilter.FilteredSafeSearch:
-		safesearch.IncWithTime(entry.Time)
+	if entry.Result != nil {
+		if entry.Result.IsFiltered {
+			filtered.IncWithTime(entry.Time)
+		}
+
+		switch entry.Result.Reason {
+		case dnsfilter.NotFilteredWhiteList:
+			whitelisted.IncWithTime(entry.Time)
+		case dnsfilter.NotFilteredError:
+			errorsTotal.IncWithTime(entry.Time)
+		case dnsfilter.FilteredBlackList:
+			filteredLists.IncWithTime(entry.Time)
+		case dnsfilter.FilteredSafeBrowsing:
+			filteredSafebrowsing.IncWithTime(entry.Time)
+		case dnsfilter.FilteredParental:
+			filteredParental.IncWithTime(entry.Time)
+		case dnsfilter.FilteredInvalid:
+			// do nothing
+		case dnsfilter.FilteredSafeSearch:
+			safesearch.IncWithTime(entry.Time)
+		}
 	}
 	elapsedTime.ObserveWithTime(entry.Elapsed.Seconds(), entry.Time)
 }