diff options
author | Michael Orlitzky <mjo@gentoo.org> | 2017-11-02 13:27:17 -0400 |
---|---|---|
committer | Michael Orlitzky <mjo@gentoo.org> | 2017-11-04 19:37:21 -0400 |
commit | 6a5bac28672abbc54f164d853d929a3e4d8b654d (patch) | |
tree | 97a9b1291eb6d90fd13e6e748b5646ca71cb875e /net-analyzer/pnp4nagios | |
parent | net-analyzer/pnp4nagios: new revision with a better fix for CVE-2012-3457. (diff) | |
download | gentoo-6a5bac28672abbc54f164d853d929a3e4d8b654d.tar.gz gentoo-6a5bac28672abbc54f164d853d929a3e4d8b654d.tar.bz2 gentoo-6a5bac28672abbc54f164d853d929a3e4d8b654d.zip |
net-analyzer/pnp4nagios: new revision to fix insecure config permissions.
Previous revisions of pnp4nagios install /etc/pnp owned by the "nagios
user," and the npcd daemon also runs as that user. That configuration
is insecure: the unprivileged user can edit /etc/pnp/npcd.cfg, and
escalate his own privileges by setting "user = root". To avoid the
problem, we set INSTALL_OPTS="" while running "emake install". That
leaves all of /etc/pnp with the default (root:root) ownership.
Bug: https://github.com/lingej/pnp4nagios/issues/140
Package-Manager: Portage-2.3.8, Repoman-2.3.3
Diffstat (limited to 'net-analyzer/pnp4nagios')
-rw-r--r-- | net-analyzer/pnp4nagios/pnp4nagios-0.6.26-r3.ebuild (renamed from net-analyzer/pnp4nagios/pnp4nagios-0.6.26-r2.ebuild) | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/net-analyzer/pnp4nagios/pnp4nagios-0.6.26-r2.ebuild b/net-analyzer/pnp4nagios/pnp4nagios-0.6.26-r3.ebuild index 818bc3104ffe..ce886f53738a 100644 --- a/net-analyzer/pnp4nagios/pnp4nagios-0.6.26-r2.ebuild +++ b/net-analyzer/pnp4nagios/pnp4nagios-0.6.26-r3.ebuild @@ -68,7 +68,9 @@ src_compile() { } src_install() { - emake DESTDIR="${D}" install install-config + # Don't use INSTALL_OPTS because they set insecure permissions on + # all of /etc/pnp (https://github.com/lingej/pnp4nagios/issues/140). + emake INSTALL_OPTS="" DESTDIR="${D}" install install-config einstalldocs newinitd "${FILESDIR}"/npcd.initd npcd rm "${ED%/}/usr/share/pnp/install.php" || \ |