LibC: Remove a few comments now that we have man pages for this.

This commit is contained in:
Nico Weber 2020-06-30 16:36:06 -04:00 committed by Andreas Kling
parent 1f323076f0
commit cbbd55bd6b
Notes: sideshowbarker 2024-07-19 05:12:30 +09:00

View file

@ -2149,8 +2149,6 @@ int Process::sys$seteuid(uid_t euid)
{ {
REQUIRE_PROMISE(id); REQUIRE_PROMISE(id);
// This has FreeBSD semantics.
// Linux and Solaris also allow m_euid.
if (euid != m_uid && euid != m_suid && !is_superuser()) if (euid != m_uid && euid != m_suid && !is_superuser())
return -EPERM; return -EPERM;
@ -2162,8 +2160,6 @@ int Process::sys$setegid(gid_t egid)
{ {
REQUIRE_PROMISE(id); REQUIRE_PROMISE(id);
// This has FreeBSD semantics.
// Linux and Solaris also allow m_egid.
if (egid != m_gid && egid != m_sgid && !is_superuser()) if (egid != m_gid && egid != m_sgid && !is_superuser())
return -EPERM; return -EPERM;
@ -2175,13 +2171,9 @@ int Process::sys$setuid(uid_t uid)
{ {
REQUIRE_PROMISE(id); REQUIRE_PROMISE(id);
// Linux and Solaris require real or saved.
// FreeBSD requires real or effective.
if (uid != m_uid && uid != m_euid && !is_superuser()) if (uid != m_uid && uid != m_euid && !is_superuser())
return -EPERM; return -EPERM;
// Solaris and Linux only set uid and suid if is_superuser(),
// FreeBSD always sets all 3.
m_uid = uid; m_uid = uid;
m_euid = uid; m_euid = uid;
m_suid = uid; m_suid = uid;
@ -2192,13 +2184,9 @@ int Process::sys$setgid(gid_t gid)
{ {
REQUIRE_PROMISE(id); REQUIRE_PROMISE(id);
// Linux and Solaris require real or saved.
// FreeBSD requires real or effective.
if (gid != m_gid && gid != m_egid && !is_superuser()) if (gid != m_gid && gid != m_egid && !is_superuser())
return -EPERM; return -EPERM;
// Solaris and Linux only set uid and suid if is_superuser(),
// FreeBSD always sets all 3.
m_gid = gid; m_gid = gid;
m_egid = gid; m_egid = gid;
m_sgid = gid; m_sgid = gid;