Merge pull request #41081 from thaJeztah/19.03_backport_fix_sandbox_cleanup

AkihiroSuda · web-flow · commit 1490b1df515b · 2020-07-26T16:17:32.000+09:00
[19.03 backport] allocateNetwork: fix network sandbox not cleaned up on failure
Upstream-commit: 2d4bfdc789ad0bb3e7718af06bdc44b42a999433
Component: engine
diff --git a/components/engine/daemon/container_operations.go b/components/engine/daemon/container_operations.go
@@ -509,7 +509,7 @@ func (daemon *Daemon) updateContainerNetworkSettings(container *container.Contai
 	}
 }
 
-func (daemon *Daemon) allocateNetwork(container *container.Container) error {
+func (daemon *Daemon) allocateNetwork(container *container.Container) (retErr error) {
 	start := time.Now()
 	controller := daemon.netController
 
@@ -577,7 +577,7 @@ func (daemon *Daemon) allocateNetwork(container *container.Container) error {
 			}
 			updateSandboxNetworkSettings(container, sb)
 			defer func() {
-				if err != nil {
+				if retErr != nil {
 					sb.Delete()
 				}
 			}()

Original file line number	Diff line number	Diff line change
`@@ -509,7 +509,7 @@ func (daemon Daemon) updateContainerNetworkSettings(container container.Contai`
`509`	`509`	`}`
`510`	`510`	`}`
`511`	`511`
`512`		`-func (daemon Daemon) allocateNetwork(container container.Container) error {`
	`512`	`+func (daemon Daemon) allocateNetwork(container container.Container) (retErr error) {`
`513`	`513`	`start := time.Now()`
`514`	`514`	`controller := daemon.netController`
`515`	`515`
`@@ -577,7 +577,7 @@ func (daemon Daemon) allocateNetwork(container container.Container) error {`
`577`	`577`	`}`
`578`	`578`	`updateSandboxNetworkSettings(container, sb)`
`579`	`579`	`defer func() {`
`580`		`- if err != nil {`
	`580`	`+ if retErr != nil {`
`581`	`581`	`sb.Delete()`
`582`	`582`	`}`
`583`	`583`	`}()`