Repository navigation
Conversation
Each claim was re-read against f34fe27; what held is recorded with file:line and a priority, what did not (login.conf read after jail_attach, lenient jid in the fw-lease dispatcher) is left out. - §2: mount_nullfs source above the tenant root (MED); byPath answers from stale rows (LOW); ipfw verbs could be gated by the leased slot (LOW, hardening). - §5: `${VAR:-default}` unsupported — webapp.yml / postgres.yml and three stack files broken (MED); the `.local` key policy can be sidestepped (MED); `devices:` merge by shape, hostname trailing dot (LOW). - §7: etc/keep on existing files, run/user HOME under root, profile rebuild by mtime, profile-from-trace and O_DIRECTORY, audit false WARNs, backup/export streams 0644 through symlinks (MED); replicate wedging, signal exit codes, wrap Exec quoting, LineBuffer, raw non-UTF-8 in JSON (LOW). - §11: command paths outside pathnames.h (LOW). No code change, no version bump. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01X6t6tzVypHye5bDGLxzmZK
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
TODO-only: no code change, no version bump.
I re-read every remaining MED/LOW claim from the bug hunt against
f34fe27. The ones that held are recorded withfile:lineand a priority. Two did not hold and are left out:jail_attach— this matchesjexec -l/ login(1), and only rlimits, priority and umask are applied.§2 crated privops / HTTP — security
mount_nullfssource that lies above the tenant root passessourceForeign. A tenant can bind/var/runinto its own jail and reach every other tenant's rootfs. trust-model.md says this was closed in 1.1.17.findOwnerByPathanswers from stale rows;byJid/byNamedon't.§5 Spec parsing
MED:
${VAR:-default}is not supported. As a result:examples/webapp.ymldoes not parse;postgres.ymlhands pkg a literal package name;CI validates only
nginx.yml.MED: the
.localforbidden-key list can be sidestepped. The parser accepts hyphenated synonyms the list does not check,securelevelsits at the wrong path, a nested duplicatesharegoes unchecked, anddevices/ipc/raw_sockets/zfs/datasetsare not listed.LOW:
devices:merges by shape, not by device class.LOW: a hostname with a trailing dot is accepted by the spec but refused by privops.
§7 Lifecycle, storage, GUI
etc/keepfails on files that already exist in the jail's /etc (hosts,resolv.conf).run/userunder a root run keeps HOME=/root and puts the X cookie in/root.profile-from-tracetreatsO_DIRECTORYopens as file reads.crate auditfalse WARNs:ncwithout-v, and bpf classed as a host device.-F, and it destroys the local snapshot on any error);§11 Build
lib/pathnames.h, listed one by one.Generated by Claude Code