basic/log: use getenv instead of secure_getenv
authorZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Wed, 5 Jul 2017 03:54:00 +0000 (23:54 -0400)
committerZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Wed, 5 Jul 2017 04:16:50 +0000 (00:16 -0400)
commita99e002cc863c27008a60490cdf76b69b9efb8c1
tree05bc3da8d2839c25e25940b925906da52da0d707
parent5ce497b59a99bb851d8e2907e9e3a633ccedba9b
basic/log: use getenv instead of secure_getenv

secure_getenv does not work when the process has a nonempty permitted
capability set, which means that it's unduly hard to configure logging in
systemd-logind, systemd-resolved, and others.

secure_getenv is useful for code in libraries which might get called from a
setuid application. log_parse_environment() is never called from our library
code, but directly form various top-level executables. None of them are
installed suid, and none are prepared to be used this way, since many
additional changes would be required to make that safe. We may just as well
drop the check and allow SYSTEMD_LOG_* to properly parsed.

Fixes #4900.
src/basic/log.c