From: Rhonda D'Vine <rhonda@proxmox.com>
Date: Fri, 4 Jan 2019 10:34:13 +0000 (+0100)
Subject: debianpkg: use getent instead of egrepping files
X-Git-Tag: frr-7.1-dev~41^2
X-Git-Url: https://git.puffer.fish/?a=commitdiff_plain;h=a9433f2a4eaf7cbe50525ea7b64248f377c44d3b;p=mirror%2Ffrr.git

debianpkg: use getent instead of egrepping files

User data might not be stored in the files in etc. getent is the
dedicated tool to extract those information, regardless of where the
user data is stored

Signed-off-by: Rhonda D'Vine <rhonda@proxmox.com>
---

diff --git a/debianpkg/frr.postinst b/debianpkg/frr.postinst
index 32af741c98..a8d6ab2805 100644
--- a/debianpkg/frr.postinst
+++ b/debianpkg/frr.postinst
@@ -1,16 +1,13 @@
 #!/bin/bash -e
 
 ######################
-PASSWDFILE=/etc/passwd
-GROUPFILE=/etc/group
+frruid=`getent passwd frr | awk -F ":" '{ print $3 }'`
+frrgid=`getent group frr | awk -F ":" '{ print $3 }'`
+frrvtygid=`getent group frrvty | awk -F ":" '{ print $3 }'`
 
-frruid=`egrep "^frr:" $PASSWDFILE | awk -F ":" '{ print $3 }'`
-frrgid=`egrep "^frr:" $GROUPFILE | awk -F ":" '{ print $3 }'`
-frrvtygid=`egrep "^frrvty:" $GROUPFILE | awk -F ":" '{ print $3 }'`
-
-[ -n ${frruid} ]    || (echo "No uid for frr in ${PASSWDFILE}"   && /bin/false)
-[ -n ${frrgid} ]    || (echo "No gid for frr in ${GROUPFILE}"    && /bin/false)
-[ -n ${frrVTYgid} ] || (echo "No gid for frrvty in ${GROUPFILE}" && /bin/false)
+[ -n ${frruid} ]    || (echo "No uid for frr"    && /bin/false)
+[ -n ${frrgid} ]    || (echo "No gid for frr"    && /bin/false)
+[ -n ${frrVTYgid} ] || (echo "No gid for frrvty" && /bin/false)
 
 chown ${frruid}:${frrgid} /etc/frr
 chown ${frruid}:${frrgid} /etc/frr/*