From: Richard Lewis <richard.lewis.debian@googlemail.com>
Date: Thu, 17 Oct 2024 22:22:21 +0100
Subject: chkrootkit: chk_echo

Forwarded: yes
(Forwarded by email: 21 Dec 2024)
---
 chkrootkit | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/chkrootkit b/chkrootkit
index 5707890..a84f8d5 100755
--- a/chkrootkit
+++ b/chkrootkit
@@ -2107,7 +2107,10 @@ chk_date () {
 
 chk_echo () {
     STATUS=${NOT_INFECTED}
-    CMD=`loc echo echo $pth`
+    CMD=$(loc echo echo "$pth")
+    if [ "${?}" -ne 0 ]; then
+        return "${NOT_FOUND}"
+    fi
 
     if [ "${EXPERT}" = "t" ]; then
         expertmode_output "${strings} -a ${CMD}"
@@ -2115,15 +2118,15 @@ chk_echo () {
         return 5
     fi
 
-    if ${strings} -a ${CMD} | ${egrep} "${GENERIC_ROOTKIT_LABEL}" > /dev/null 2>&1
+    if "${strings}" -a "${CMD}" | ${egrep} "${GENERIC_ROOTKIT_LABEL}" > /dev/null 2>&1
     then
         STATUS=${INFECTED}
     fi
-    if ${ls} -l ${CMD} | ${egrep} "^...s" > /dev/null 2>&1
+    if "${ls}" -l "${CMD}" | ${egrep} "^...s" > /dev/null 2>&1
     then
         STATUS=${INFECTED}
     fi
-    return ${STATUS}
+    return "${STATUS}"
 }
 
 chk_env () {
