From a29335823fc4a41f93d5d6a6b334478ade1cadb8 Mon Sep 17 00:00:00 2001
From: mboelen <michael@cisofy.com>
Date: Thu, 16 Apr 2015 20:00:09 +0200
Subject: [PATCH] Improved core dump checking

---
 include/tests_kernel | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/include/tests_kernel b/include/tests_kernel
index 64e74aa2..a2ba9e9b 100644
--- a/include/tests_kernel
+++ b/include/tests_kernel
@@ -383,12 +383,12 @@
         if [ -f /etc/security/limits.conf ]; then
             logtext "Result: file /etc/security/limits.conf exists" 
             logtext "Test: Checking if core dumps are disabled in /etc/security/limits.conf"
-            FIND1=`cat /etc/security/limits.conf | grep -v "^#" | grep -v "^$" | awk '{ if ($1=="*" && $2=="soft" && $3=="core") { print "soft core enabled" } }'`
-            FIND2=`cat /etc/security/limits.conf | grep -v "^#" | grep -v "^$" | awk '{ if ($1=="*" && $2=="hard" && $3=="core") { print "hard core enabled" } }'`
+            FIND1=`cat /etc/security/limits.conf | grep -v "^#" | grep -v "^$" | awk '{ if ($1=="*" && $2=="soft" && $3=="core" && $4=="1") { print "soft core enabled" } }'`
+            FIND2=`cat /etc/security/limits.conf | grep -v "^#" | grep -v "^$" | awk '{ if ($1=="*" && $2=="hard" && $3=="core" && $4=="1") { print "hard core enabled" } }'`
             if [ "${FIND1}" = "soft core enabled" -o "${FIND2}" = "hard core enabled" ]; then
                 logtext "Result: core dumps (soft or hard) are enabled"
                 Display --indent 2 --text "- Checking core dumps configuration" --result ENABLED --color YELLOW
-                #YYY suggestion
+                AddSuggestion "${TEST_NO}" "Check if core dumps need to be enabled on this system"
                 AddHP 1 2
               else
                 logtext "Result: core dumps (soft and hard) are both disabled"