beenull/moby
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Change switch to if so that the break statements correctly breaks loop

Browse source 
Docker-DCO-1.1-Signed-off-by: Kohei Tsuruta <coheyxyz@gmail.com> (github: coheyxyz)
This commit is contained in:
Kohei Tsuruta 2014-07-16 23:09:17 +09:00
parent 3b3f0fa085
commit ffe885ec0e
1 changed files with 2 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
daemon/networkdriver/bridge/driver.go
View file

@ -415,8 +415,7 @@ func AllocatePort(job *engine.Job) engine.Status {
break
}
switch allocerr := err.(type) {
case portallocator.ErrPortAlreadyAllocated:
if allocerr, ok := err.(portallocator.ErrPortAlreadyAllocated); ok {
// There is no point in immediately retrying to map an explicitly
// chosen port.
if hostPort != 0 {
@ -426,7 +425,7 @@ func AllocatePort(job *engine.Job) engine.Status {
// Automatically chosen 'free' port failed to bind: move on the next.
job.Logf("Failed to bind %s for container address %s. Trying another port.", allocerr.IPPort(), container.String())
default:
} else {
// some other error during mapping
job.Logf("Received an unexpected error during port allocation: %s", err.Error())
break