diff options
author | Giuseppe Scrivano <gscrivan@redhat.com> | 2022-09-14 13:03:56 +0200 |
---|---|---|
committer | Giuseppe Scrivano <gscrivan@redhat.com> | 2022-09-14 17:08:57 +0200 |
commit | 92dc61d5edb1b5ce85f7e4563d400cc861a28359 (patch) | |
tree | ae03dc1172ae575bbc27da284a11ca6a6f55ad62 | |
parent | 017d81ddd0e8d228aadb949175c0aae1e4b9d925 (diff) | |
download | podman-92dc61d5edb1b5ce85f7e4563d400cc861a28359.tar.gz podman-92dc61d5edb1b5ce85f7e4563d400cc861a28359.tar.bz2 podman-92dc61d5edb1b5ce85f7e4563d400cc861a28359.zip |
libpod: rename function
the function checks if a path is under any mount, not just bind
mounts.
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
-rw-r--r-- | libpod/container_internal_common.go | 4 | ||||
-rw-r--r-- | libpod/container_path_resolution.go | 4 |
2 files changed, 4 insertions, 4 deletions
diff --git a/libpod/container_internal_common.go b/libpod/container_internal_common.go index c7f59aba5..9c4a3bb67 100644 --- a/libpod/container_internal_common.go +++ b/libpod/container_internal_common.go @@ -531,7 +531,7 @@ func (c *Container) isWorkDirSymlink(resolvedPath string) bool { } if resolvedSymlink != "" { _, resolvedSymlinkWorkdir, err := c.resolvePath(c.state.Mountpoint, resolvedSymlink) - if isPathOnVolume(c, resolvedSymlinkWorkdir) || isPathOnBindMount(c, resolvedSymlinkWorkdir) { + if isPathOnVolume(c, resolvedSymlinkWorkdir) || isPathOnMount(c, resolvedSymlinkWorkdir) { // Resolved symlink exists on external volume or mount return true } @@ -564,7 +564,7 @@ func (c *Container) resolveWorkDir() error { // If the specified workdir is a subdir of a volume or mount, // we don't need to do anything. The runtime is taking care of // that. - if isPathOnVolume(c, workdir) || isPathOnBindMount(c, workdir) { + if isPathOnVolume(c, workdir) || isPathOnMount(c, workdir) { logrus.Debugf("Workdir %q resolved to a volume or mount", workdir) return nil } diff --git a/libpod/container_path_resolution.go b/libpod/container_path_resolution.go index 35622d623..eddfd361e 100644 --- a/libpod/container_path_resolution.go +++ b/libpod/container_path_resolution.go @@ -152,9 +152,9 @@ func findBindMount(c *Container, containerPath string) *specs.Mount { return nil } -/// isPathOnBindMount returns true if the specified containerPath is a subdir of any +/// isPathOnMount returns true if the specified containerPath is a subdir of any // Mount's destination. -func isPathOnBindMount(c *Container, containerPath string) bool { +func isPathOnMount(c *Container, containerPath string) bool { cleanedContainerPath := filepath.Clean(containerPath) for _, m := range c.config.Spec.Mounts { if cleanedContainerPath == filepath.Clean(m.Destination) { |