container-storage-setup icon indicating copy to clipboard operation
container-storage-setup copied to clipboard

Mount leak in platform_supports_deferred_deletion()

Open edsantiago opened this issue 9 years ago • 5 comments

Line 141:

mount -o bind $TEMPDIR $TEMPDIR

...but lines 143 & 148 return without umounting. System is then left with stuck mounts:

/dev/vda1 on /tmp/tmp.vBynTZ78If type xfs (rw,relatime,seclabel,attr2,inode64,noquota)

edsantiago avatar Nov 01 '16 20:11 edsantiago

Ed, in your case which line is returning without mounting? 143 or 148?

rhvgoyal avatar Nov 01 '16 20:11 rhvgoyal

It's actually 143, because of what I can only assume is a packaging bug: /usr/lib/docker{,-latest}-storage-setup/dss-child-read-write does not seem to exist. I'm 99% sure, once I get to 100% I'll file a bz

edsantiago avatar Nov 01 '16 20:11 edsantiago

cc @lsm5

rhvgoyal avatar Nov 01 '16 20:11 rhvgoyal

Regardless of which return is at fault, both should be fixed (add umount before each one)

edsantiago avatar Nov 01 '16 20:11 edsantiago

yep, I am fixing that now

rhvgoyal avatar Nov 01 '16 20:11 rhvgoyal