From 15170735badf9c8c8f13840efcb70bf7ba192b4d Mon Sep 17 00:00:00 2001 From: Ophestra Date: Sun, 3 Aug 2025 18:06:41 +0900 Subject: container/mount: move tmpfs sysroot prefixing to caller The mountTmpfs helper is a relatively low level function that is not exposed as part of the API. Prefixing sysroot here not only introduces overhead but is also quite error-prone. Signed-off-by: Ophestra --- container/ops.go | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) (limited to 'container/ops.go') diff --git a/container/ops.go b/container/ops.go index b25f3209..b8d287fa 100644 --- a/container/ops.go +++ b/container/ops.go @@ -190,13 +190,12 @@ type MountDevOp string func (d MountDevOp) early(*Params) error { return nil } func (d MountDevOp) apply(params *Params) error { v := string(d) - if !path.IsAbs(v) { return msg.WrapErr(EBADE, fmt.Sprintf("path %q is not absolute", v)) } target := toSysroot(v) - if err := mountTmpfs(SourceTmpfsDevtmpfs, v, MS_NOSUID|MS_NODEV, 0, params.ParentPerm); err != nil { + if err := mountTmpfs(SourceTmpfsDevtmpfs, target, MS_NOSUID|MS_NODEV, 0, params.ParentPerm); err != nil { return err } @@ -332,7 +331,7 @@ func (t *MountTmpfsOp) apply(*Params) error { if t.Size < 0 || t.Size > math.MaxUint>>1 { return msg.WrapErr(EBADE, fmt.Sprintf("size %d out of bounds", t.Size)) } - return mountTmpfs(t.FSName, t.Path, t.Flags, t.Size, t.Perm) + return mountTmpfs(t.FSName, toSysroot(t.Path), t.Flags, t.Size, t.Perm) } func (t *MountTmpfsOp) Is(op Op) bool { vt, ok := op.(*MountTmpfsOp); return ok && *t == *vt } -- cgit v1.3.1