kotovalexarian-likes-github/moby--moby
1
0
Fork 0
mirror of https://github.com/moby/moby.git synced 2022-11-09 12:21:53 -05:00
Code Releases Activity

Only check if route overlaps routes with scope: LINK

Browse source 
Signed-off-by: Alex Nordlund <alexander.nordlund@nasdaq.com>
This commit is contained in:
Alex Nordlund 2021-07-05 16:54:13 +02:00
parent 42d2048b9d
commit ee9e526764
2 changed files with 10 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

2
libnetwork/netutils/utils_linux.go
View file

@ -31,7 +31,7 @@ func CheckRouteOverlaps(toCheck *net.IPNet) error {
return err return err
} }
for _, network := range networks { for _, network := range networks {
if network.Dst != nil && NetworkOverlaps(toCheck, network.Dst) { if network.Dst != nil && network.Scope == netlink.SCOPE_LINK && NetworkOverlaps(toCheck, network.Dst) {
return ErrNetworkOverlaps return ErrNetworkOverlaps
} }
} }

10
libnetwork/netutils/utils_linux_test.go
View file

@ -46,8 +46,11 @@ func TestCheckRouteOverlaps(t *testing.T) {
routes := []netlink.Route{} routes := []netlink.Route{}
for _, addr := range routesData { for _, addr := range routesData {
_, netX, _ := net.ParseCIDR(addr) _, netX, _ := net.ParseCIDR(addr)
routes = append(routes, netlink.Route{Dst: netX}) routes = append(routes, netlink.Route{Dst: netX, Scope: netlink.SCOPE_LINK})
} }
// Add a route with a scope which should not overlap
_, netX, _ := net.ParseCIDR("10.0.5.0/24")
routes = append(routes, netlink.Route{Dst: netX, Scope: netlink.SCOPE_UNIVERSE})
return routes, nil return routes, nil
} }
defer func() { networkGetRoutesFct = nil }() defer func() { networkGetRoutesFct = nil }()
@ -61,6 +64,11 @@ func TestCheckRouteOverlaps(t *testing.T) {
if err := CheckRouteOverlaps(netX); err == nil { if err := CheckRouteOverlaps(netX); err == nil {
t.Fatal("10.0.2.0/24 and 10.0.2.0 should overlap but it doesn't") t.Fatal("10.0.2.0/24 and 10.0.2.0 should overlap but it doesn't")
} }
_, netX, _ = net.ParseCIDR("10.0.5.0/24")
if err := CheckRouteOverlaps(netX); err != nil {
t.Fatal("10.0.5.0/24 and 10.0.5.0 with scope UNIVERSE should not overlap but it does")
}
} }
func TestCheckNameserverOverlaps(t *testing.T) { func TestCheckNameserverOverlaps(t *testing.T) {