]> sourceware.org Git - newlib-cygwin.git/commit
Cygwin: fix: seteuid32() must return EPERM if privileges are not held.
authorJ.H. van de Water <houder@xs4all.nl>
Wed, 27 Mar 2019 16:01:03 +0000 (17:01 +0100)
committerCorinna Vinschen <corinna@vinschen.de>
Wed, 27 Mar 2019 16:08:49 +0000 (17:08 +0100)
commitf46c9ab46c46cd2bf02eb5e5af591e19b1c5a10a
tree6941d3d29fafa04c953dc6f8112f8e3595552658
parenta137da74bae803a4770dce9a6f38f1def582fe80
Cygwin: fix: seteuid32() must return EPERM if privileges are not held.

Starting w/ the intro of S4U, seteuid32() calls lsaprivkeyauth(), then
s4uauth(). s4uauth calls LsaRegisterLogonProcess().
LsaRegisterLogonProcess fails w/ STATUS_PORT_CONNECTION_REFUSED, if the
proper privileges are not held.
Because of RtlNtStatusToDosError(), this status would be mapped to
ERROR_ACCESS_DENIED, which in turn would map to EACCES. Therefore it is
useless to add this status to errmap[] (errno.cc), as s4auauth() should
return EPERM as errno here (i.e. if process is not privileged).

Hence the kludge.

Before the intro of S4U, seteuid32() called lsaprivkeyauth(), then
lsaauth(), then create_token(). Before the intro of Vista, the latter
would have called NtCreateToken().
NtCreateToken() would have failed w/ STATUS_PRIVILEGE_NOT_HELD for a
process w/o the proper privileges. In that case, calling seteuid32()
would have returned EPERM (as required).

Since the intro of Vista, and if the process had been started from an
UNelevated shell, create_token() does NOT reach NtCreateToken()!
As create_token() failed to properly set errno in that case, calling
seteuid32() would return errno as set by lsaauth(), i.e. EACCES, not
in agreement w/ Posix (a bug which was present for years).
(lsaauth() called LsaRegisterLogonProcess() which would fail)
winsup/cygwin/sec_auth.cc
This page took 0.029661 seconds and 5 git commands to generate.