git: 2e031fd0719b - main - cred: supplementary_group_member() => group_is_supplementary()
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Sat, 02 Nov 2024 20:39:24 UTC
The branch main has been updated by olce:
URL: https://cgit.FreeBSD.org/src/commit/?id=2e031fd0719b567e66198bc76c8a1c6c9b7f2920
commit 2e031fd0719b567e66198bc76c8a1c6c9b7f2920
Author: Olivier Certner <olce@FreeBSD.org>
AuthorDate: 2024-07-16 20:55:53 +0000
Commit: Olivier Certner <olce@FreeBSD.org>
CommitDate: 2024-11-02 20:37:35 +0000
cred: supplementary_group_member() => group_is_supplementary()
The new name seems more immediately descriptive.
While here, constify its signature ('struct ucred' isn't modified).
While here, replace "supplemental" => "supplementary" in some comments.
No functional change (intended).
Reviewed by: mhorne, emaste
Approved by: markj (mentor)
MFC after: 3 days
Differential Revision: https://reviews.freebsd.org/D46906
---
sys/kern/kern_prot.c | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/sys/kern/kern_prot.c b/sys/kern/kern_prot.c
index 95362ce8a310..4668bdea3fdd 100644
--- a/sys/kern/kern_prot.c
+++ b/sys/kern/kern_prot.c
@@ -1280,13 +1280,13 @@ sys___setugid(struct thread *td, struct __setugid_args *uap)
* Returns whether gid designates a supplementary group in cred.
*/
static bool
-supplementary_group_member(gid_t gid, struct ucred *cred)
+group_is_supplementary(const gid_t gid, const struct ucred *const cred)
{
int l, h, m;
/*
- * Perform a binary search of the supplemental groups. This is possible
- * because we sort the groups in crsetgroups().
+ * Perform a binary search of the supplementary groups. This is
+ * possible because we sort the groups in crsetgroups().
*/
l = 1;
h = cred->cr_ngroups;
@@ -1320,7 +1320,7 @@ groupmember(gid_t gid, struct ucred *cred)
if (gid == cred->cr_groups[0])
return (true);
- return (supplementary_group_member(gid, cred));
+ return (group_is_supplementary(gid, cred));
}
/*
@@ -1333,7 +1333,7 @@ realgroupmember(gid_t gid, struct ucred *cred)
if (gid == cred->cr_rgid)
return (true);
- return (supplementary_group_member(gid, cred));
+ return (group_is_supplementary(gid, cred));
}
/*
@@ -2316,7 +2316,7 @@ crextend(struct ucred *cr, int n)
/*
* Copy groups in to a credential, preserving any necessary invariants.
- * Currently this includes the sorting of all supplemental gids.
+ * Currently this includes the sorting of all supplementary gids.
* crextend() must have been called before hand to ensure sufficient
* space is available.
*/