Details

Reviewers

mjg
kib
dchagin
pauamma_gundo.com
olce

Commits

rG6345a69ee271: p_candebug(9): cr_bsd_visible() impacts, misc fixes
rGdaf8ad192942: p_candebug(9): cr_bsd_visible() impacts, misc fixes
rG76781950658c: p_candebug(9): cr_bsd_visible() impacts, misc fixes
rGeb94f24fab4b: p_candebug(9): cr_bsd_visible() impacts, misc fixes

Summary

Mention cr_bsd_visible(9). Remove references to cr_canseeothergids(9) and
cr_canseeotheruids(9), as well as indirect references not immediately useful.

Fix description of credentials checks to match reality.

Re-order errors to match code's check order.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 52739
Build 49630: arc lint + arc unit

Event Timeline

olce created this revision.Jun 20 2023, 1:45 PM

Herald added a subscriber: imp. · View Herald TranscriptJun 20 2023, 1:45 PM

olce requested review of this revision.Jun 20 2023, 1:45 PM

Harbormaster completed remote builds in B52173: Diff 123504.Jun 20 2023, 1:45 PM

olce added a parent revision: D40637: p_cansee(9): Bring up-to-date, misc fixes.Jun 20 2023, 2:11 PM

olce added a child revision: D40639: prison_check(9): Bring up-to-date with hierarchical jails.

olce added reviewers: mjg, kib, dchagin.Jun 20 2023, 9:03 PM

emaste added subscribers: manpages, emaste.Jun 29 2023, 3:58 PM

bcr added a subscriber: bcr.Jun 29 2023, 4:05 PM

bcr added inline comments.

share/man/man9/p_candebug.9
79	missing a comma here at the end?

olce added inline comments.Jun 29 2023, 7:28 PM

share/man/man9/p_candebug.9
79	Not sure I understand. Are you suggesting: An unprivileged process attempted to debug another process, but the system is ? But if you really mean after `deny it`, then a priori I don't think so since these words are followed by a parenthetical clause and a dot after it.