Page MenuHomeFreeBSD
Differential D52257

procfs: Restore printing the effective GID in 'status'
ClosedPublic
Actions

Authored by olce on Aug 29 2025, 11:02 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 7, 11:06 PM
Unknown Object (File)
Sun, Dec 7, 5:45 AM
Unknown Object (File)
Wed, Nov 26, 1:18 PM
Unknown Object (File)
Thu, Nov 20, 11:17 AM
Unknown Object (File)
Nov 2 2025, 4:27 AM
Unknown Object (File)
Oct 31 2025, 7:15 PM
Unknown Object (File)
Oct 25 2025, 1:32 PM
Unknown Object (File)
Oct 25 2025, 3:00 AM
View All 26 Files
Subscribers
emaste
imp

Details

Reviewers
kevans
kib
Commits
rG79c1160fb9f2: procfs: Restore printing the effective GID in 'status'
rGd859d4c57eda: procfs: Restore printing the effective GID in 'status'
Summary

Fixes: be1f7435ef218b1d ("kern: start tracking cr_gid outside of cr_groups[]")
Sponsored by: The FreeBSD Foundation

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

olce created this revision.Aug 29 2025, 11:02 PM
Herald added a subscriber: imp. · View Herald TranscriptAug 29 2025, 11:02 PM
olce requested review of this revision.Aug 29 2025, 11:02 PM
Harbormaster completed remote builds in B66660: Diff 161223.Aug 29 2025, 11:02 PM
olce added a reviewer: kib.Sep 1 2025, 3:39 PM
kib accepted this revision.Sep 1 2025, 4:01 PM
This revision is now accepted and ready to land.Sep 1 2025, 4:01 PM
kib added a comment.Sep 1 2025, 4:02 PM

Does linprocfs have the same issue? I mean compat/linprocfs/linprocfs.c:1136

olce added a comment.Sep 1 2025, 4:21 PM
In D52257#1194563, @kib wrote:

Does linprocfs have the same issue? I mean compat/linprocfs/linprocfs.c:1136

It doesn't as it already has been printing cr_gid separately, and then the content of cr_groups[].

That also means that before it would output the effective GID twice, but after our changes, the latter will disappear from the "Groups" part if not also a supplementary group. This is actually a "free" bugfix, as Linux does not output the effective GID twice.

Closed by commit rGd859d4c57eda: procfs: Restore printing the effective GID in 'status' (authored by olce). · Explain WhySep 9 2025, 3:58 PM
This revision was automatically updated to reflect the committed changes.
olce added a commit: rGd859d4c57eda: procfs: Restore printing the effective GID in 'status'.
olce added a commit: rG79c1160fb9f2: procfs: Restore printing the effective GID in 'status'.Sep 18 2025, 8:53 AM

Revision Contents
Changeset List

PathSize
sys/
fs/
procfs/
8 lines

Diff 161806

View Options

sys/fs/procfs/procfs_status.c

Loading...