OpenBSD: Fix parameters in call to afs_close()
[openafs.git] / src / afs / OBSD / osi_groups.c
index 73aef9d..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,15 +47,19 @@ Afs_xsetgroups(p, args, retval)
     AFS_STATCNT(afs_xsetgroups);
     AFS_GLOCK();
 
-    code = afs_InitReq(&treq, osi_curcred());
+    p = osi_curproc();
+
+    code = afs_InitReq(&treq, p->p_rcred);
     AFS_GUNLOCK();
-    if (code) return code;
+    if (code)
+       return code;
 
     code = setgroups(p, args, retval);
-    /* Note that if there is a pag already in the new groups we don't
+    /*
+     * Note that if there is a pag already in the new groups we don't
      * overwrite it with the old pag.
      */
-    if (PagInCred(osi_curcred()) == NOPAG) {
+    if (PagInCred(p->p_rcred) == NOPAG) {
        if (((treq.uid >> 24) & 0xff) == 'A') {
            AFS_GLOCK();
            /* we've already done a setpag, so now we redo it */
@@ -77,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;
@@ -85,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;
@@ -103,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;
@@ -120,37 +123,27 @@ 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)
 {
-    gid_t *gp;
-    struct ucred *newcr, *cr;
+    struct ucred *cr = *cred;
+    int i;
 
     AFS_STATCNT(afs_setgroups);
-    /*
-     * The real setgroups() call does this, so maybe we should too.
-     *
-     */
+
     if (ngroups > NGROUPS)
        return EINVAL;
-    cr = *cred;
-    if (!change_parent) {
-       crhold(cr);
-       newcr = crcopy(cr);
-    } else
-       newcr = cr;
-    newcr->cr_ngroups = ngroups;
-    gp = newcr->cr_groups;
-    while (ngroups--)
-       *gp++ = *gidset++;
-    for ( ; gp < &(*cred)->cr_groups[NGROUPS]; gp++)
-       *gp = NOGROUP;
-    *cred = newcr;
-    return(0);
+
+    if (!change_parent)
+       cr = crcopy(cr);
+
+    for (i = 0; i < ngroups; i++)
+       cr->cr_groups[i] = gidset[i];
+    for (i = ngroups; i < NGROUPS; i++)
+       cr->cr_groups[i] = NOGROUP;
+    cr->cr_ngroups = ngroups;
+
+    *cred = cr;
+    return (0);
 }