diff options
| author | Roland McGrath <roland@gnu.org> | 1995-12-22 10:00:21 +0000 |
|---|---|---|
| committer | Roland McGrath <roland@gnu.org> | 1995-12-22 10:00:21 +0000 |
| commit | 975320984b9f9fa6742136447ec82e7e304d83fd (patch) | |
| tree | 6638c381441825ac81f4f9355eba25fd8863c7f9 /sysdeps/mach/hurd/setreuid.c | |
| parent | e3fa2641f4f296230aa2607f0df9d71ab3f9c7af (diff) | |
| download | glibc-cvs/libc-951222.tar.gz glibc-cvs/libc-951222.tar.xz glibc-cvs/libc-951222.zip | |
Fri Dec 22 00:57:38 1995 Roland McGrath <roland@churchy.gnu.ai.mit.edu> cvs/libc-951222
* 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.
Diffstat (limited to 'sysdeps/mach/hurd/setreuid.c')
| -rw-r--r-- | sysdeps/mach/hurd/setreuid.c | 44 |
1 files changed, 35 insertions, 9 deletions
diff --git a/sysdeps/mach/hurd/setreuid.c b/sysdeps/mach/hurd/setreuid.c index 1dcbf0ee82..530e4a20c7 100644 --- a/sysdeps/mach/hurd/setreuid.c +++ b/sysdeps/mach/hurd/setreuid.c @@ -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)); |