OpenBSD: Fix parameters in call to afs_close()
[openafs.git] / src / afs / OBSD / osi_groups.c
index c170b1e..e26088f 100644 (file)
 #include <afsconfig.h>
 #include "afs/param.h"
 
-RCSID("$Header$");
 
 #include "afs/sysincludes.h"
 #include "afs/afsincludes.h"
-#include "afs/afs_stats.h"  /* statistics */
+#include "afs/afs_stats.h"     /* statistics */
 #include "sys/syscallargs.h"
 
 #define NOUID   ((uid_t) -1)
@@ -30,24 +29,17 @@ RCSID("$Header$");
 
 
 static int
-afs_getgroups(
-    struct ucred *cred,
-    int ngroups,
-    gid_t *gidset);
+  afs_getgroups(struct ucred *cred, int ngroups, gid_t * gidset);
 
 static int
-afs_setgroups(
-    struct proc *proc,
-    struct ucred **cred,
-    int ngroups,
-    gid_t *gidset,
-    int change_parent);
+  afs_setgroups(struct proc *proc, struct ucred **cred, int ngroups,
+               gid_t * gidset, int change_parent);
 
 int
 Afs_xsetgroups(p, args, retval)
-    struct proc *p;
-    void *args;
-    int *retval;
+     struct proc *p;
+     void *args;
+     int *retval;
 {
     int code = 0;
     struct vrequest treq;
@@ -55,6 +47,8 @@ Afs_xsetgroups(p, args, retval)
     AFS_STATCNT(afs_xsetgroups);
     AFS_GLOCK();
 
+    p = osi_curproc();
+
     code = afs_InitReq(&treq, p->p_rcred);
     AFS_GUNLOCK();
     if (code)
@@ -79,7 +73,7 @@ Afs_xsetgroups(p, args, retval)
 
 int
 setpag(struct proc *proc, struct ucred **cred, afs_uint32 pagvalue,
-       afs_uint32 *newpag, int change_parent)
+       afs_uint32 * newpag, int change_parent)
 {
     gid_t gidset[NGROUPS];
     int ngroups, code;
@@ -87,17 +81,27 @@ setpag(struct proc *proc, struct ucred **cred, afs_uint32 pagvalue,
 
     AFS_STATCNT(setpag);
     ngroups = afs_getgroups(*cred, NGROUPS, gidset);
+    /*
+     * If the group list is empty, use the task's primary group as the group
+     * list. Otherwise, when setting the PAG, group 0 will be set to arbitrary
+     * gibberish and the PAG, which starts at group offset 1, will not be
+     * properly set because the group count will be wrong (2 instead of 3).
+     */
+    if (ngroups == 0) {
+       gidset[0] = (*cred)->cr_gid;
+       ngroups = 1;
+    }
     if (afs_get_pag_from_groups(gidset[1], gidset[2]) == NOPAG) {
        /* We will have to shift grouplist to make room for pag */
        if (ngroups + 2 > NGROUPS) {
            return (E2BIG);
        }
-       for (j = ngroups -1; j >= 0; j--) {
-           gidset[j+2] = gidset[j];
-       }
+       for (j = ngroups - 1; j >= 0; j--) {
+           gidset[j + 2] = gidset[j];
+       }
        ngroups += 2;
     }
-    *newpag = (pagvalue == -1 ? genpag(): pagvalue);
+    *newpag = (pagvalue == -1 ? genpag() : pagvalue);
     afs_get_groups_from_pag(*newpag, &gidset[1], &gidset[2]);
     code = afs_setgroups(proc, cred, ngroups, gidset, change_parent);
     return code;
@@ -105,10 +109,7 @@ setpag(struct proc *proc, struct ucred **cred, afs_uint32 pagvalue,
 
 
 static int
-afs_getgroups(
-    struct ucred *cred,
-    int ngroups,
-    gid_t *gidset)
+afs_getgroups(struct ucred *cred, int ngroups, gid_t * gidset)
 {
     int ngrps, savengrps;
     gid_t *gp;
@@ -123,12 +124,8 @@ afs_getgroups(
 
 
 static int
-afs_setgroups(
-    struct proc *proc,
-    struct ucred **cred,
-    int ngroups,
-    gid_t *gidset,
-    int change_parent)
+afs_setgroups(struct proc *proc, struct ucred **cred, int ngroups,
+             gid_t * gidset, int change_parent)
 {
     struct ucred *cr = *cred;
     int i;
@@ -148,5 +145,5 @@ afs_setgroups(
     cr->cr_ngroups = ngroups;
 
     *cred = cr;
-    return(0);
+    return (0);
 }