bytecodealliance / bytecodealliance/cap-std

Linux: support RESOLVE_NO_XDEV when openat2 is unavailable

Offen
#423 5 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Rust
Sterne
821
Forks
57
Ø Merge
1 Std. 16 Min.
Gemergte PRs (30 T.)
4

Beschreibung

**TL;DR:** On Linux, cap-primitives falls back to its manual path resolver when
`openat2()` is unavailable, but the fallback cannot enforce
`RESOLVE_NO_XDEV`. Downstream users therefore cannot safely reject mount
crossings on systems where `openat2()` returns `ENOSYS`.

**Context:** This surfaced through bootc's `open_dir_noxdev()` usage under QEMU
linux-user, but the missing behavior is general and is not specific to bootc or
QEMU. The downstream report is
https://github.com/bootc-dev/bootc/issues/1481.

## Current behavior

`cap-primitives/src/rustix/linux/fs/open_impl.rs` uses `openat2()` for its fast
path and falls back to `manually::open()` when the syscall is unavailable. The
manual resolver preserves capability containment and resolves symlinks one
component at a time, but it has no `NO_XDEV` policy.

As a result, cap-std-ext implements `open_dir_noxdev()` with a direct
`openat2()` call. It handles `EXDEV`, but propagates `ENOSYS` instead of using a
capability-safe fallback.

## Reproduction

| Component | Value |
| ----------------- | ---------------------- |
| Host and VM | x86_64, Fedora 44 |
| Container | aarch64 Fedora bootc |
| Affected emulator | QEMU linux-user 8.2.9 |
| Working emulator | QEMU linux-user 10.2.2 |
| cap-primitives | 4.0.3 |
| cap-std-ext | 5.1.2 |

With QEMU 8.2.9 registered through `binfmt_misc`:

```console
$ podman run --rm --arch arm64 \
quay.io/fedora/fedora-bootc:latest \
bootc container lint
error: Linting: Function not implemented (os error 38)
```

The trace reports unsupported AArch64 syscall 437 immediately before the
failure. That syscall is `openat2`, same image passes under QEMU 10.2.2,
which implements `openat2`.

## Required behavior

The fallback must reject crossing any mount, including same filesystem bind
mounts and mounts reached through symlinks, while retaining the manual
resolver's containment and race-resistance properties.

A final `st_dev` comparison is insufficient because a bind mount may have the
same device and inode as its source. Checking only the final descriptor is also
insufficient: an intermediate component or symlink may cross a mount even when
the final component is not a mount root. Each opened component must therefore
be checked before symlink interpretation or descent.

If reliable mount information is unavailable, the operation should return an
explicit unsupported error rather than silently perform a normal open.

## Proposed direction

1. Keep `openat2()` as the fast path and add `RESOLVE_NO_XDEV` when requested.
2. On `ENOSYS`, reuse the existing manual component resolver rather than add a
separate path walker.
3. Check every newly opened descriptor for a mount crossing before following a
symlink or descending into that component.
4. Return `EXDEV` for a detected crossing and fail closed when mount identity
cannot be determined reliably.
5. Expose this as an existing-directory operation through the public layer the
maintainers consider appropriate, such as `cap-fs-ext`. Existing opens
should remain unchanged when the policy is not requested.

`STATX_ATTR_MOUNT_ROOT` appears suitable for the fallback, under QEMU 8.2.9 it
correctly identified all 153 mount points visible in `/proc/self/mountinfo`, a
same filesystem bind mount, and descriptors held across mount and detach
operations. `STATX_MNT_ID` was not reliable in this environment because QEMU
reported the field as available but returned zero for tested paths.

The initial API should be limited to opening existing directories, applying a
general `OpenOptions` flag to effectful operations such as truncation would
require a broader design because validating the result after opening is too
late.

## References

- Downstream report: https://github.com/bootc-dev/bootc/issues/1481
- `openat2(2)`: https://man7.org/linux/man-pages/man2/openat2.2.html
- `statx(2)`: https://man7.org/linux/man-pages/man2/statx.2.html
- cap-std-ext API: https://docs.rs/cap-std-ext/latest/cap_std_ext/dirext/trait.CapStdExtDirExt.html

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Start with cap-primitives/src/rustix/linux/fs/open_impl.rs and the existing manually::open() resolver, then trace cap-std-ext’s open_dir_noxdev() path and the public cap-fs-ext API. Verify the fallback checks each opened component before symlink traversal or descent, returns EXDEV or an explicit unsupported error when required, and leaves opens without the policy unchanged.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
linux, rust
Bereich
operating-systems, security
Issue-Typ
Feature
Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Aktivitätsstatus
Aktiv
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.