[Bug 272093] The 'see_other_gids' security policy considers the effective group IDs and not the real ones
Date: Thu, 28 Sep 2023 15:11:18 UTC
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=272093
--- Comment #1 from commit-hook@FreeBSD.org ---
A commit in branch main references this bug:
URL:
https://cgit.FreeBSD.org/src/commit/?id=91658080f1a598ddda03943a783c9a941199f7d2
commit 91658080f1a598ddda03943a783c9a941199f7d2
Author: Olivier Certner <olce.freebsd@certner.fr>
AuthorDate: 2023-08-17 23:54:45 +0000
Commit: Mitchell Horne <mhorne@FreeBSD.org>
CommitDate: 2023-09-28 15:05:46 +0000
cr_canseeothergids(): Use real instead of effective group membership
Using the effective group and not the real one when testing membership
has the consequence that unprivileged processes cannot see setuid
commands they launch until these have relinquished their privileges.
This is also in contradiction with how the similar cr_canseeotheruids()
works, i.e., by taking into account real user IDs.
Fix this by substituting groupmember() with realgroupmember(). While
here, simplify the code.
PR: 272093
Reviewed by: mhorne
MFC after: 2 weeks
Sponsored by: Kumacom SAS
Differential Revision: https://reviews.freebsd.org/D40642
sys/kern/kern_prot.c | 23 ++++++++++-------------
1 file changed, 10 insertions(+), 13 deletions(-)
--
You are receiving this mail because:
You are the assignee for the bug.