Fri Dec 22 00:57:38 1995 Roland McGrath <roland@churchy.gnu.ai.mit.edu> cvs/libc-951222
authorRoland McGrath <roland@gnu.org>
Fri, 22 Dec 1995 10:00:21 +0000 (10:00 +0000)
committerRoland McGrath <roland@gnu.org>
Fri, 22 Dec 1995 10:00:21 +0000 (10:00 +0000)
* sysdeps/mach/hurd/setuid.c: Rewrote ID frobnication to do the
right thing.
* sysdeps/mach/hurd/setgid.c: Likewise.
* sysdeps/mach/hurd/setreuid.c: Likewise.
* sysdeps/mach/hurd/setregid.c: Likewise.
* sysdeps/mach/hurd/setegid.c: Likewise.
* sysdeps/mach/hurd/seteuid.c: Likewise.

* sysdeps/mach/hurd/fork.c: Peek __mach_task_self_ value before
proc_dostop call to work around kernel paging bug.

Thu Dec 21 12:19:32 1995  Miles Bader  <miles@gnu.ai.mit.edu>

* sysdeps/mach/hurd/setuid.c (__setuid): Actually add the new uid
  instead of putting it (and other uids) in the gids and leaving the
  old uids as is.

ChangeLog
sysdeps/mach/hurd/setegid.c
sysdeps/mach/hurd/seteuid.c
sysdeps/mach/hurd/setgid.c
sysdeps/mach/hurd/setregid.c
sysdeps/mach/hurd/setreuid.c
sysdeps/mach/hurd/setuid.c

index 66f7e41..d2b8241 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,22 @@
+Fri Dec 22 00:57:38 1995  Roland McGrath  <roland@churchy.gnu.ai.mit.edu>
+
+       * sysdeps/mach/hurd/setuid.c: Rewrote ID frobnication to do the
+       right thing.
+       * sysdeps/mach/hurd/setgid.c: Likewise.
+       * sysdeps/mach/hurd/setreuid.c: Likewise.
+       * sysdeps/mach/hurd/setregid.c: Likewise.
+       * sysdeps/mach/hurd/setegid.c: Likewise.
+       * sysdeps/mach/hurd/seteuid.c: Likewise.
+
+       * sysdeps/mach/hurd/fork.c: Peek __mach_task_self_ value before
+       proc_dostop call to work around kernel paging bug.
+
+Thu Dec 21 12:19:32 1995  Miles Bader  <miles@gnu.ai.mit.edu>
+
+       * sysdeps/mach/hurd/setuid.c (__setuid): Actually add the new uid
+       instead of putting it (and other uids) in the gids and leaving the
+       old uids as is.
+
 Wed Dec 20 18:23:10 1995  Roland McGrath  <roland@churchy.gnu.ai.mit.edu>
 
        * stdio/internals.c (flushbuf): If the target is -1, always
index 8ca4abe..6710462 100644 (file)
@@ -1,4 +1,4 @@
-/* Copyright (C) 1993, 1994 Free Software Foundation, Inc.
+/* Copyright (C) 1993, 1994, 1995 Free Software Foundation, Inc.
 This file is part of the GNU C Library.
 
 The GNU C Library is free software; you can redistribute it and/or
@@ -40,17 +40,18 @@ DEFUN(setegid, (gid), gid_t gid)
       /* Make a new auth handle which has EGID as the first element in the
          list of effective gids.  */
 
-      size_t ngen = _hurd_id.gen.ngids < 1 ? 1 : _hurd_id.gen.ngids;
-      gid_t newgen[ngen];
-
-      newgen[0] = gid;
-      memcpy (&newgen[1], _hurd_id.gen.gids, (ngen - 1) * sizeof (gid_t));
+      if (_hurd_id.gen.ngids > 0)
+       {
+         _hurd_id.gen.gids[0] = gid;
+         _hurd_id.valid = 0;
+       }
 
       err = __USEPORT (AUTH, __auth_makeauth
                       (port, NULL, MACH_MSG_TYPE_COPY_SEND, 0,
                        _hurd_id.gen.uids, _hurd_id.gen.nuids,
                        _hurd_id.aux.uids, _hurd_id.aux.nuids,
-                       newgen, ngen,
+                       _hurd_id.gen.ngids ? _hurd_id.gen.gids : &gid,
+                       _hurd_id.gen.ngids ?: 1,
                        _hurd_id.aux.gids, _hurd_id.aux.ngids,
                        &newauth));
     }
index 9c44c4d..62f4488 100644 (file)
@@ -1,4 +1,4 @@
-/* Copyright (C) 1993, 1994 Free Software Foundation, Inc.
+/* Copyright (C) 1993, 1994, 1995 Free Software Foundation, Inc.
 This file is part of the GNU C Library.
 
 The GNU C Library is free software; you can redistribute it and/or
@@ -40,15 +40,16 @@ DEFUN(seteuid, (uid), uid_t uid)
       /* Make a new auth handle which has EUID as the first element in the
          list of effective uids.  */
 
-      size_t ngen = _hurd_id.gen.nuids < 1 ? 1 : _hurd_id.gen.nuids;
-      uid_t newgen[ngen];
-
-      newgen[0] = uid;
-      memcpy (&newgen[1], _hurd_id.gen.uids, (ngen - 1) * sizeof (uid_t));
+      if (_hurd_id.gen.nuids > 0)
+       {
+         _hurd_id.gen.uids[0] = uid;
+         _hurd_id.valid = 0;
+       }
 
       err = __USEPORT (AUTH, __auth_makeauth
                       (port, NULL, MACH_MSG_TYPE_COPY_SEND, 0,
-                       newgen, ngen,
+                       _hurd_id.gen.nuids ? _hurd_id.gen.uids : &uid,
+                       _hurd_id.gen.nuids ?: 1,
                        _hurd_id.aux.uids, _hurd_id.aux.nuids,
                        _hurd_id.gen.gids, _hurd_id.gen.ngids,
                        _hurd_id.aux.gids, _hurd_id.aux.ngids,
index 933792b..660a3fe 100644 (file)
@@ -43,22 +43,45 @@ DEFUN(__setgid, (gid), gid_t gid)
       /* Make a new auth handle which has GID as the real gid,
         and as the first element in the list of effective gids.  */
 
-      gid_t newgen[_hurd_id.gen.ngids + 1];
-      gid_t newaux[_hurd_id.aux.ngids];
-      
-      newgen[0] = gid;
-      memcpy (&newgen[1], _hurd_id.gen.gids,
-             _hurd_id.gen.ngids * sizeof (gid_t));
-      newaux[0] = gid;
-      memcpy (&newaux[1], _hurd_id.aux.gids,
-             (_hurd_id.aux.ngids - 1) * sizeof (gid_t));
+      gid_t *newgen, *newaux, auxbuf[2];
+      size_t ngen, naux;
+
+      newaux = _hurd_id.aux.gids;
+      naux = _hurd_id.aux.ngids;
+      if (_hurd_id.gen.ngids == 0)
+       {
+         /* No effective gids now.  The new set will be just GID.  */
+         newgen = &gid;
+         ngen = 1;
+       }
+      else if (_hurd_id.gen.gids[0] == 0)
+       {
+         /* We are root; set the effective, real, and saved to GID.  */
+         _hurd_id.gen.gids[0] = gid;
+         _hurd_id.valid = 0;
+         newgen = _hurd_id.gen.gids;
+         ngen = _hurd_id.gen.ngids;
+         if (_hurd_id.aux.ngids < 2)
+           {
+             newaux = auxbuf;
+             naux = 2;
+           }
+         _hurd_id.aux.gids[0] = _hurd_id.aux.gids[1] = gid;
+       }
+      else
+       {
+         /* We are not root; just change the effective GID.  */
+         _hurd_id.gen.gids[0] = gid;
+         _hurd_id.valid = 0;
+         newgen = _hurd_id.gen.gids;
+         ngen = _hurd_id.gen.ngids;
+       }
 
       err = __USEPORT (AUTH, __auth_makeauth
                       (port, NULL, MACH_MSG_TYPE_COPY_SEND, 0,
-                       _hurd_id.gen.uids, _hurd_id.gen.nuids,
-                       _hurd_id.aux.uids, _hurd_id.aux.nuids,
-                       newgen, 1 + _hurd_id.gen.ngids,
-                       newaux, _hurd_id.aux.ngids,
+                       newgen, ngen, newaux, naux,
+                       _hurd_id.gen.gids, _hurd_id.gen.ngids,
+                       _hurd_id.aux.gids, _hurd_id.aux.ngids,
                        &newauth));
     }
   __mutex_unlock (&_hurd_id.lock);
index 8b76f70..ea7e38b 100644 (file)
@@ -39,21 +39,47 @@ __setregid (gid_t rgid, gid_t egid)
       /* Make a new auth handle which has RGID as the real gid,
         and EGID as the first element in the list of effective gids.  */
 
-      size_t ngen = _hurd_id.gen.ngids < 1 ? 1 : _hurd_id.gen.ngids;
-      size_t naux = _hurd_id.aux.ngids < 1 ? 1 : _hurd_id.aux.ngids;
-      gid_t newaux[naux], newgen[ngen];
+      gid_t *newgen, *newaux;
+      size_t ngen, naux;
 
-      newgen[0] = egid;
-      memcpy (&newgen[1], _hurd_id.gen.gids, (ngen - 1) * sizeof (gid_t));
-      newaux[0] = rgid;
-      memcpy (&newaux[1], _hurd_id.aux.gids, (naux - 1) * sizeof (gid_t));
+      newgen = _hurd_id.gen.gids;
+      ngen = _hurd_id.gen.ngids;
+      if (egid != -1)
+       {
+         if (_hurd_id.gen.ngids == 0)
+           {
+             /* No effective gids now.  The new set will be just GID.  */
+             newgen = &egid;
+             ngen = 1;
+           }
+         else
+           {
+             _hurd_id.gen.gids[0] = egid;
+             _hurd_id.valid = 0;
+           }
+       }
+
+      newaux = _hurd_id.aux.gids;
+      naux = _hurd_id.aux.ngids;
+      if (rgid != -1)
+       {
+         if (_hurd_id.aux.ngids == 0)
+           {
+             newaux = &rgid;
+             naux = 1;
+           }
+         else
+           {
+             _hurd_id.aux.gids[0] = rgid;
+             _hurd_id.valid = 0;
+           }
+       }
 
       err = __USEPORT (AUTH, __auth_makeauth
                       (port, NULL, MACH_MSG_TYPE_COPY_SEND, 0,
-                       _hurd_id.gen.gids, _hurd_id.gen.ngids,
-                       _hurd_id.aux.gids, _hurd_id.aux.ngids,
-                       newgen, ngen,
-                       newaux, naux,
+                       _hurd_id.gen.uids, _hurd_id.gen.nuids,
+                       _hurd_id.aux.uids, _hurd_id.aux.nuids,
+                       newgen, ngen, newaux, naux,
                        &newauth));
     }
   __mutex_unlock (&_hurd_id.lock);
index 1dcbf0e..530e4a2 100644 (file)
@@ -39,19 +39,45 @@ __setreuid (uid_t ruid, uid_t euid)
       /* Make a new auth handle which has RUID as the real uid,
         and EUID as the first element in the list of effective uids.  */
 
-      size_t ngen = _hurd_id.gen.nuids < 1 ? 1 : _hurd_id.gen.nuids;
-      size_t naux = _hurd_id.aux.nuids < 1 ? 1 : _hurd_id.aux.nuids;
-      uid_t newaux[naux], newgen[ngen];
+      uid_t *newgen, *newaux;
+      size_t ngen, naux;
 
-      newgen[0] = euid;
-      memcpy (&newgen[1], _hurd_id.gen.uids, (ngen - 1) * sizeof (uid_t));
-      newaux[0] = ruid;
-      memcpy (&newaux[1], _hurd_id.aux.uids, (naux - 1) * sizeof (uid_t));
+      newgen = _hurd_id.gen.uids;
+      ngen = _hurd_id.gen.nuids;
+      if (euid != -1)
+       {
+         if (_hurd_id.gen.nuids == 0)
+           {
+             /* No effective uids now.  The new set will be just UID.  */
+             newgen = &euid;
+             ngen = 1;
+           }
+         else
+           {
+             _hurd_id.gen.uids[0] = euid;
+             _hurd_id.valid = 0;
+           }
+       }
+
+      newaux = _hurd_id.aux.uids;
+      naux = _hurd_id.aux.nuids;
+      if (ruid != -1)
+       {
+         if (_hurd_id.aux.nuids == 0)
+           {
+             newaux = &ruid;
+             naux = 1;
+           }
+         else
+           {
+             _hurd_id.aux.uids[0] = ruid;
+             _hurd_id.valid = 0;
+           }
+       }
 
       err = __USEPORT (AUTH, __auth_makeauth
                       (port, NULL, MACH_MSG_TYPE_COPY_SEND, 0,
-                       newgen, ngen,
-                       newaux, naux,
+                       newgen, ngen, newaux, naux,
                        _hurd_id.gen.gids, _hurd_id.gen.ngids,
                        _hurd_id.aux.gids, _hurd_id.aux.ngids,
                        &newauth));
index 70d5ba4..586ad21 100644 (file)
@@ -43,20 +43,43 @@ DEFUN(__setuid, (uid), uid_t uid)
       /* Make a new auth handle which has UID as the real uid,
         and as the first element in the list of effective uids.  */
 
-      uid_t newgen[_hurd_id.gen.nuids + 1];
-      uid_t newaux[_hurd_id.aux.nuids];
-      
-      newgen[0] = uid;
-      memcpy (&newgen[1], _hurd_id.gen.uids,
-             _hurd_id.gen.nuids * sizeof (uid_t));
-      newaux[0] = uid;
-      memcpy (&newaux[1], _hurd_id.aux.uids + 1,
-             (_hurd_id.aux.nuids - 1) * sizeof (uid_t));
+      uid_t *newgen, *newaux, auxbuf[2];
+      size_t ngen, naux;
+
+      newaux = _hurd_id.aux.uids;
+      naux = _hurd_id.aux.nuids;
+      if (_hurd_id.gen.nuids == 0)
+       {
+         /* No effective uids now.  The new set will be just UID.  */
+         newgen = &uid;
+         ngen = 1;
+       }
+      else if (_hurd_id.gen.uids[0] == 0)
+       {
+         /* We are root; set the effective, real, and saved to UID.  */
+         _hurd_id.gen.uids[0] = uid;
+         _hurd_id.valid = 0;
+         newgen = _hurd_id.gen.uids;
+         ngen = _hurd_id.gen.nuids;
+         if (_hurd_id.aux.nuids < 2)
+           {
+             newaux = auxbuf;
+             naux = 2;
+           }
+         _hurd_id.aux.uids[0] = _hurd_id.aux.uids[1] = uid;
+       }
+      else
+       {
+         /* We are not root; just change the effective UID.  */
+         _hurd_id.gen.uids[0] = uid;
+         _hurd_id.valid = 0;
+         newgen = _hurd_id.gen.uids;
+         ngen = _hurd_id.gen.nuids;
+       }
 
       err = __USEPORT (AUTH, __auth_makeauth
                       (port, NULL, MACH_MSG_TYPE_COPY_SEND, 0,
-                       newgen, 1 + _hurd_id.gen.nuids,
-                       newaux, _hurd_id.aux.nuids,
+                       newgen, ngen, newaux, naux,
                        _hurd_id.gen.gids, _hurd_id.gen.ngids,
                        _hurd_id.aux.gids, _hurd_id.aux.ngids,
                        &newauth));