opensource/podman
1
0
Fork 0
mirror of https://github.com/containers/podman.git synced 2025-07-04 18:27:33 +08:00
Code Issues Packages Projects Releases Wiki Activity

libpod: Factor out usage of unix.MNT_DETACH from (*Volume).unmount

Browse Source 
There is an existing wrapper for unix.Unmount(..., MNT_DETACH) in
util_linux.go but that filters all errors and for volumes, we only want
to filter EINVAL. The existing libpod.Unmount seems to only have one
call site so perhaps these can be merged.

[NO NEW TESTS NEEDED]

Signed-off-by: Doug Rabson <dfr@rabson.org>
This commit is contained in:
Doug Rabson
2022-09-20 11:30:12 +01:00
parent 9de2a5ff79
commit abe8dad344
2 changed files with 10 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
libpod/volume_internal_common.go
View File

@ -180,7 +180,7 @@ func (v *Volume) unmount(force bool) error {
} }
// Unmount the volume // Unmount the volume
if err := unix.Unmount(v.config.MountPoint, unix.MNT_DETACH); err != nil { if err := detachUnmount(v.config.MountPoint); err != nil {
if err == unix.EINVAL { if err == unix.EINVAL {
// Ignore EINVAL - the mount no longer exists. // Ignore EINVAL - the mount no longer exists.
return nil return nil

9
libpod/volume_internal_linux.go Normal file
View File

@ -0,0 +1,9 @@
package libpod
import (
"golang.org/x/sys/unix"
)
func detachUnmount(mountPoint string) error {
return unix.Unmount(mountPoint, unix.MNT_DETACH)
}