From c1650ae415ad4f1075c24e791a547382600ed798 Mon Sep 17 00:00:00 2001
From: Bob Mottram <bob@freedombone.net>
Date: Sat, 5 Aug 2017 23:07:31 +0100
Subject: [PATCH] Only update limits when needed

---
 src/freedombone-utils-setup | 14 ++++++++++----
 1 file changed, 10 insertions(+), 4 deletions(-)

diff --git a/src/freedombone-utils-setup b/src/freedombone-utils-setup
index c89624045..fd772f62b 100755
--- a/src/freedombone-utils-setup
+++ b/src/freedombone-utils-setup
@@ -473,14 +473,18 @@ function limit_user_logins {
     if ! grep -q '* hard maxsyslogins' /etc/security/limits.conf; then
         echo '* hard maxsyslogins 10' >> /etc/security/limits.conf
     else
-        sed -i 's|hard maxsyslogins.*|hard maxsyslogins 10|g' /etc/security/limits.conf
+        if ! grep -q '* hard maxsyslogins 10' /etc/security/limits.conf; then
+            sed -i 's|hard maxsyslogins.*|hard maxsyslogins 10|g' /etc/security/limits.conf
+        fi
     fi
 
     # Max logins for each user
     if ! grep -q '* hard maxlogins' /etc/security/limits.conf; then
         echo '* hard maxlogins 2' >> /etc/security/limits.conf
     else
-        sed -i 's|hard maxlogins.*|hard maxlogins 2|g' /etc/security/limits.conf
+        if ! grep -q '* hard maxlogins 2' /etc/security/limits.conf; then
+            sed -i 's|hard maxlogins.*|hard maxlogins 2|g' /etc/security/limits.conf
+        fi
     fi
 }
 
@@ -609,10 +613,12 @@ function lockdown_permissions {
 }
 
 function disable_core_dumps {
-    if ! grep -q '* hard core 0' /etc/security/limits.conf; then
+    if ! grep -q '* hard core' /etc/security/limits.conf; then
         echo '* hard core 0' >> /etc/security/limits.conf
     else
-        sed -i 's|hard core.*|hard core 0|g' /etc/security/limits.conf
+        if ! grep -q '* hard core 0' /etc/security/limits.conf; then
+            sed -i 's|hard core.*|hard core 0|g' /etc/security/limits.conf
+        fi
     fi
 }
 
-- 
GitLab