From 5d71cc01b6bb089a70fa1e855943dab0d88439bb Mon Sep 17 00:00:00 2001
From: Alessandro Boch <aboch@docker.com>
Date: Mon, 6 Feb 2017 16:37:51 -0800
Subject: [PATCH] getAllNetworks unnecessary walks all the networks

- libnetwork controller Networks() already returns
  a copy list. Also Networks() correctly skips any
  network which ahs already been marked for deletion
  while getNetworks implementation bypass this.

Signed-off-by: Alessandro Boch <aboch@docker.com>
---
 daemon/network.go | 10 +---------
 1 file changed, 1 insertion(+), 9 deletions(-)

diff --git a/daemon/network.go b/daemon/network.go
index d01e5e8719..8e42824ba8 100644
--- a/daemon/network.go
+++ b/daemon/network.go
@@ -96,15 +96,7 @@ func (daemon *Daemon) GetNetworksByID(partialID string) []libnetwork.Network {
 
 // getAllNetworks returns a list containing all networks
 func (daemon *Daemon) getAllNetworks() []libnetwork.Network {
-	c := daemon.netController
-	list := []libnetwork.Network{}
-	l := func(nw libnetwork.Network) bool {
-		list = append(list, nw)
-		return false
-	}
-	c.WalkNetworks(l)
-
-	return list
+	return daemon.netController.Networks()
 }
 
 func isIngressNetwork(name string) bool {