Fix: Panic (from IP.Parse())

If the IP address is neither v4 or v6, IP.Parse() returns "nil".

Related: #64

collector/rbl.go

@@ -146,7 +146,15 @@ func (rbl *Rbl) lookup(rblList string, targetHost string) []Rblresult {
 		res.Address = ip
 		res.Rbl = rblList
 
-		revIP := godnsbl.Reverse(net.ParseIP(ip))
+		// attempt to "validate" the IP
+		ValidIPAddress := net.ParseIP(ip)
+		if ValidIPAddress == nil {
+			log.Errorf("Unable to parse IP: %s", ip)
+			continue
+		}
+
+		// reverse it, for the look up
+		revIP := godnsbl.Reverse(ValidIPAddress)
 
 		rbl.query(revIP, rblList, &res)
 

collector/rbl_test.go

@@ -0,0 +1,20 @@
+package collector
+
+import (
+	"os"
+	"testing"
+
+	log "github.com/sirupsen/logrus"
+)
+
+func TestUpdate(t *testing.T) {
+	log.SetLevel(log.DebugLevel)
+	log.SetOutput(os.Stdout)
+
+	rbl := NewRbl("0.0.0.0:53")
+	rbl.Update("this.is.not.an.ip", []string{"cbl.abuseat.org"})
+
+	if len(rbl.Results) > 0 {
+		t.Errorf("Got a result, but shouldn't have: %v", rbl.Results)
+	}
+}