From 6c7ae06435d6e288024691f1133d7a2a24fd8ef3 Mon Sep 17 00:00:00 2001 From: Alexander Larsson Date: Thu, 19 Sep 2013 19:23:35 +0200 Subject: [PATCH] Image.Changes: Deactivate image device after unmounting it There is no need to keep the image device around if we were the onces creating the device. --- image.go | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/image.go b/image.go index f495e8e8fc..ff8b836c39 100644 --- a/image.go +++ b/image.go @@ -617,6 +617,8 @@ func (image *Image) Changes(runtime *Runtime, root, rw, id string) ([]Change, er return nil, err } + wasActivated := devices.HasActivatedDevice(image.ID) + // We re-use rw for the temporary mount of the base image as its // not used by device-mapper otherwise err = devices.MountDevice(image.ID, rw) @@ -626,6 +628,9 @@ func (image *Image) Changes(runtime *Runtime, root, rw, id string) ([]Change, er changes, err := ChangesDirs(root, rw) _ = devices.UnmountDevice(image.ID, rw) + if !wasActivated { + _ = devices.DeactivateDevice(image.ID) + } if err != nil { return nil, err }