debianpkg: Fix maintainer-script-should-not-use-recursive-chown-or-chmod

New lintian version causes warning for recursive chown and chmod. This
change avoids the the recursive operation by splitting it into to commands

Signed-off-by: Martin Winter <mwinter@opensourcerouting.org>

diff --git a/debianpkg/frr.postinst b/debianpkg/frr.postinst
index cbc45f03ebd0dfde8feac4e5684e3f1df747ca86..972f8c0500fb500fdd48004307f14e195b9e876b 100644 (file)
--- a/debianpkg/frr.postinst
+++ b/debianpkg/frr.postinst
@@ -12,7 +12,8 @@ frrvtygid=`egrep "^frrvty:" $GROUPFILE | awk -F ":" '{ print $3 }'`
 [ -n ${frrgid} ]    || (echo "No gid for frr in ${GROUPFILE}"    && /bin/false)
 [ -n ${frrVTYgid} ] || (echo "No gid for frrvty in ${GROUPFILE}" && /bin/false)
 
-chown -R ${frruid}:${frrgid} /etc/frr
+chown ${frruid}:${frrgid} /etc/frr
+chown ${frruid}:${frrgid} /etc/frr/*
 touch /etc/frr/vtysh.conf
 chgrp ${frrvtygid} /etc/frr/vtysh*
 chmod 644 /etc/frr/*

diff --git a/debianpkg/frr.preinst b/debianpkg/frr.preinst
index 467bea971834adf8f4205e2e3f7817de3d5a1626..477e690d0aac4896bf760d605d856bf590c00e55 100644 (file)
--- a/debianpkg/frr.preinst
+++ b/debianpkg/frr.preinst
@@ -39,14 +39,16 @@ if [ "$1" = "install" ]; then
   # Logfiles are group readable in case users were put into the frr group.
   d=/var/log/frr/
     mkdir -p $d
-    chown -R frr:frr $d
+    chown frr:frr $d
+    chown --quiet frr:frr $d/* | true
     chmod u=rwx,go=rx $d
     find $d -type f -print0 | xargs -0 --no-run-if-empty   chmod u=rw,g=r,o=
 
   # Strict permissions for the sockets.
   d=/var/run/frr/
     mkdir -p $d
-    chown -R frr:frr $d
+    chown frr:frr $d
+    chown --quiet frr:frr $d/* | true
     chmod u=rwx,go=rx $d
     find $d -type f -print0 | xargs -0 --no-run-if-empty   chmod u=rw,go=