diff --git a/daemon/container.go b/daemon/container.go index 7491f4ecf1..4e44621757 100644 --- a/daemon/container.go +++ b/daemon/container.go @@ -399,16 +399,16 @@ func (container *Container) Pause() error { container.Lock() defer container.Unlock() + // We cannot Pause the container which is not running + if !container.Running { + return fmt.Errorf("Container %s is not running, cannot pause a non-running container", container.ID) + } + // We cannot Pause the container which is already paused if container.Paused { return fmt.Errorf("Container %s is already paused", container.ID) } - // We cannot Pause the container which is not running - if !container.Running { - return fmt.Errorf("Container %s is not running", container.ID) - } - if err := container.daemon.execDriver.Pause(container.command); err != nil { return err } @@ -421,14 +421,14 @@ func (container *Container) Unpause() error { container.Lock() defer container.Unlock() - // We cannot unpause the container which is not paused - if !container.Paused { - return fmt.Errorf("Container %s is not paused, so what", container.ID) - } - // We cannot unpause the container which is not running if !container.Running { - return fmt.Errorf("Container %s is not running", container.ID) + return fmt.Errorf("Container %s is not running, cannot unpause a non-running container", container.ID) + } + + // We cannot unpause the container which is not paused + if !container.Paused { + return fmt.Errorf("Container %s is not paused", container.ID) } if err := container.daemon.execDriver.Unpause(container.command); err != nil {