Merge pull request #13 from libp2p/fix/filter-listing

fix filter listing

filter.go

@@ -147,6 +147,7 @@ func (fs *Filters) AddrBlocked(a ma.Multiaddr) (deny bool) {
 func (fs *Filters) Filters() (result []*net.IPNet) {
 	ffa := fs.FiltersForAction(ActionDeny)
 	for _, res := range ffa {
+		res := res // allocate a new copy
 		result = append(result, &res)
 	}
 	return result

filter_test.go

@@ -7,6 +7,32 @@ import (
 	ma "github.com/multiformats/go-multiaddr"
 )
 
+func TestFilterListing(t *testing.T) {
+	f := NewFilters()
+	expected := map[string]bool{
+		"1.2.3.0/24":  true,
+		"4.3.2.1/32":  true,
+		"fd00::/8":    true,
+		"fc00::1/128": true,
+	}
+	for cidr := range expected {
+		_, ipnet, _ := net.ParseCIDR(cidr)
+		f.AddDialFilter(ipnet)
+	}
+
+	for _, filter := range f.Filters() {
+		cidr := filter.String()
+		if expected[cidr] {
+			delete(expected, cidr)
+		} else {
+			t.Errorf("unexected filter %s", cidr)
+		}
+	}
+	for cidr := range expected {
+		t.Errorf("expected filter %s", cidr)
+	}
+}
+
 func TestFilterBlocking(t *testing.T) {
 	f := NewFilters()