Skip to content

Keep devpts mount options off root and ptmx - #15225

Merged
copybara-service[bot] merged 2 commits into
masterfrom
test/cl992446873
Oct 2, 2026
Merged

copybara-service[bot] merged 2 commits into
masterfrom
test/cl992446873

Conversation

@copybara-service

Copy link
Copy Markdown

Keep devpts mount options off root and ptmx

51a3d72 applied mode, uid and gid to replicas but still used the uid and gid options for the root directory and ptmx. Linux initializes the root as namespace root and ptmx with the mounting process's identity 1 2. Keep those owners independent of the replica options.

Consolidate SetGid into SetMode's option table and retain its group-write mode assertion. Check root, ptmx and replica ownership for default and explicit options, and keep each private mount's cleanup active.

Assisted-by: OpenAI Codex

FUTURE_COPYBARA_INTEGRATE_REVIEW=#15162 from tamird:devpts-mount-options b07ccec

51a3d72 applied mode, uid and gid to replicas but still used the uid
and gid options for the root directory and ptmx. Linux initializes the
root as namespace root and ptmx with the mounting process's identity
[1, 2]. Keep those owners independent of the replica options.

Consolidate SetGid into SetMode's option table and retain its group-write
mode assertion. Check root, ptmx and replica ownership for default and
explicit options, and keep each private mount's cleanup active.

[1]: https://github.com/torvalds/linux/blob/830b3c68c/fs/devpts/inode.c
[2]: https://github.com/torvalds/linux/blob/830b3c68c/fs/inode.c

Assisted-by: OpenAI Codex
@copybara-service copybara-service Bot added the exported Issue was exported automatically label Oct 2, 2026
@copybara-service
copybara-service Bot merged commit 8c3c0f4 into master Oct 2, 2026
1 of 3 checks passed
@copybara-service
copybara-service Bot deleted the test/cl992446873 branch October 2, 2026 20:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

exported Issue was exported automatically

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants