From 6cdf8623d52e7e4c5b5265deb5f5b1d33f2e6e95 Mon Sep 17 00:00:00 2001 From: Jana Radhakrishnan Date: Thu, 21 May 2015 18:57:17 +0000 Subject: [PATCH] Do not attempt releasing network when not attached to any network Sometimes container.cleanup() can be called from multiple paths for the same container during error conditions from monitor and regular startup path. So if the container network has been already released do not try to release it again. Signed-off-by: Jana Radhakrishnan --- daemon/container_linux.go | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/daemon/container_linux.go b/daemon/container_linux.go index 72e8409275..e74ba3537b 100644 --- a/daemon/container_linux.go +++ b/daemon/container_linux.go @@ -917,6 +917,12 @@ func (container *Container) ReleaseNetwork() { return } + // If the container is not attached to any network do not try + // to release network and generate spurious error messages. + if container.NetworkSettings.NetworkID == "" { + return + } + n, err := container.daemon.netController.NetworkByID(container.NetworkSettings.NetworkID) if err != nil { logrus.Errorf("error locating network id %s: %v", container.NetworkSettings.NetworkID, err)