Package: postgresql-common
Version: 248
Followup-For: Bug #1010291
X-Debbugs-Cc: rvandegr...@debian.org
Control: tags -1 patch

After upgrading to bookworm, I was reminded of this bug when
/var/log/postgresql's sticky bit re-appeared.  So I dug a bit more.

postgresql-common's postinst unconditionally changes owners and modes on
/var/log/postgresql.  The patch below makes it respect dpkg-statoverride.

Thanks,
Ross

diff --git a/debian/postgresql-common.postinst 
b/debian/postgresql-common.postinst
old mode 100644
new mode 100755
index 545146a..638c8b8
--- a/debian/postgresql-common.postinst
+++ b/debian/postgresql-common.postinst
@@ -65,8 +65,10 @@ Please fix this and reinstall this package." >&2

     # nicer log directory permissions
     mkdir -p /var/log/postgresql
-    chmod 1775 /var/log/postgresql
-    chown root:postgres /var/log/postgresql
+    if ! dpkg-statoverride --list /var/log/postgresl > /dev/null; then
+        chmod "$LOG_MODE" /var/log/postgresql
+        chown root:postgres /var/log/postgresql
+    fi

     # create socket directory
     [ -d /var/run/postgresql ] || \

Reply via email to