This is an automated email from the git hooks/post-receive script. guillem pushed a commit to branch master in repository dpkg.
View the commit online: https://git.dpkg.org/cgit/dpkg/dpkg.git/commit/?id=e6bc50a5ecc6743c1ad03aede8679a81d11ac713 commit e6bc50a5ecc6743c1ad03aede8679a81d11ac713 Author: Guillem Jover <[email protected]> AuthorDate: Fri Nov 29 02:26:41 2019 +0100 libdpkg: Do not change the ownership of the triggers database directory Either we are running as root:root (checked by modstatdb_open()) which means the ownership should be correct already, or we are not which means we cannot change it anyway. --- debian/changelog | 3 +++ lib/dpkg/triglib.c | 3 --- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/debian/changelog b/debian/changelog index 73fbfafc1..2b24876be 100644 --- a/debian/changelog +++ b/debian/changelog @@ -49,6 +49,9 @@ dpkg (1.20.0) UNRELEASED; urgency=medium - Allow missing status and available databases, so that they get created on write, and remove the code setting them up from the dpkg postinst. Closes: #647911 + - Do not change the ownership of the triggers database directory. Either + we are running as root:root which means the ownership should be correct + already, or we are not which means we cannot change it anyway. * Perl modules: - Dpkg::Source::Package: Verify original tarball signatures at build time. - Dpkg::BuildFlags: Add new unset() method. diff --git a/lib/dpkg/triglib.c b/lib/dpkg/triglib.c index ab9060f66..28fb74518 100644 --- a/lib/dpkg/triglib.c +++ b/lib/dpkg/triglib.c @@ -790,9 +790,6 @@ trig_incorporate(enum modstatdb_rw cstatus) if (errno != EEXIST) ohshite(_("unable to create triggers state" " directory '%.250s'"), triggersdir); - } else if (chown(triggersdir, 0, 0)) { - ohshite(_("unable to set ownership of triggers state" - " directory '%.250s'"), triggersdir); } ur = trigdef_update_start(tduf); } -- Dpkg.Org's dpkg
