From 5d6dbcfd746d5231f2b14203503160cd3c111437 Mon Sep 17 00:00:00 2001 From: Andrew Deason Date: Thu, 16 Jan 2025 23:42:45 -0600 Subject: [PATCH] AIX: Use afs_IsPagId() Commit c856bd9c87 (afs: Introduce afs_IsPagId()) converted all of our PAG id checks to use afs_IsPagId(), except for one place in AIX's osi_groups.c by mistake. Update it, so all PAG id checks now use afs_IsPagId(). Thanks to ben@huntsmans.net for pointing out the mistake. Change-Id: Ic18967da229d60b7c9fab8da565dd80645ac1c7f Reviewed-on: https://gerrit.openafs.org/16116 Reviewed-by: Cheyenne Wills Tested-by: BuildBot Reviewed-by: Ben Huntsman Reviewed-by: Andrew Deason --- src/afs/AIX/osi_groups.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/afs/AIX/osi_groups.c b/src/afs/AIX/osi_groups.c index 0a916a39f7..7652134d7c 100644 --- a/src/afs/AIX/osi_groups.c +++ b/src/afs/AIX/osi_groups.c @@ -60,7 +60,7 @@ setgroups(ngroups, gidset) credp0 = credp; if (PagInCred(credp) == NOPAG) { - if (((treq.uid >> 24) & 0xff) == 'A') { + if (afs_IsPagId(treq.uid)) { AFS_GLOCK(); AddPag(treq.uid, &credp); AFS_GUNLOCK();