mirror of
https://github.com/CISOfy/lynis.git
synced 2025-07-30 01:04:39 +02:00
Fix wildcard expansion, absolute path handling and output to stderr
This commit is contained in:
parent
7df0b8618b
commit
6f6e21add2
@ -2180,7 +2180,8 @@
|
|||||||
for I in ${FIND}; do
|
for I in ${FIND}; do
|
||||||
I=$(echo ${I} | sed 's/:space:/ /g' | sed 's/;$//' | sed 's/ #.*$//')
|
I=$(echo ${I} | sed 's/:space:/ /g' | sed 's/;$//' | sed 's/ #.*$//')
|
||||||
OPTION=$(echo ${I} | awk '{ print $1 }')
|
OPTION=$(echo ${I} | awk '{ print $1 }')
|
||||||
VALUE=$(echo ${I}| cut -d' ' -f2-)
|
# Use quotes here to prevent wildcard expansion
|
||||||
|
VALUE=$(echo "${I}"| cut -d' ' -f2-)
|
||||||
LogText "Result: found option ${OPTION} in ${CONFIG_FILE} with value '${VALUE}'"
|
LogText "Result: found option ${OPTION} in ${CONFIG_FILE} with value '${VALUE}'"
|
||||||
STORE_SETTING=1
|
STORE_SETTING=1
|
||||||
case ${OPTION} in
|
case ${OPTION} in
|
||||||
@ -2303,9 +2304,21 @@
|
|||||||
done
|
done
|
||||||
if [ ${FOUND} -eq 0 ]; then NGINX_CONF_FILES_ADDITIONS="${NGINX_CONF_FILES_ADDITIONS} ${VALUE}"; fi
|
if [ ${FOUND} -eq 0 ]; then NGINX_CONF_FILES_ADDITIONS="${NGINX_CONF_FILES_ADDITIONS} ${VALUE}"; fi
|
||||||
# Check for additional config files included as follows
|
# Check for additional config files included as follows
|
||||||
# "include sites-enabled/*.conf"
|
# "include sites-enabled/*.conf" (relative path)
|
||||||
elif [ $(echo ${VALUE} | grep -F -c "*.conf") -gt 0 ]; then
|
# "include /etc/nginx/sites-enabled/*.conf" (absolute path)
|
||||||
for FOUND_CONF in $(ls ${CONFIG_FILE%nginx.conf}${VALUE%;*}); do
|
elif [ $(echo "${VALUE}" | grep -F -c "*.conf") -gt 0 ]; then
|
||||||
|
# Check if path is absolute or relative
|
||||||
|
case $VALUE in
|
||||||
|
/*)
|
||||||
|
# Absolute path, so list files directly from that path
|
||||||
|
CONF_LS=$(${LSBINARY} ${VALUE%;*} 2>/dev/null) # Will error if wildcard doesn't match anything, so pipe stderr to /dev/null
|
||||||
|
;;
|
||||||
|
*)
|
||||||
|
# Relative path, so construct absolute path first to list files for
|
||||||
|
CONF_LS=$(${LSBINARY} ${CONFIG_FILE%nginx.conf}${VALUE%;*} 2>/dev/null)
|
||||||
|
;;
|
||||||
|
esac
|
||||||
|
for FOUND_CONF in CONF_LS; do
|
||||||
FOUND=0
|
FOUND=0
|
||||||
for CONF in ${NGINX_CONF_FILES}; do
|
for CONF in ${NGINX_CONF_FILES}; do
|
||||||
if [ "${CONF}" = "${FOUND_CONF}" ]; then FOUND=1; LogText "Found this file already in our configuration files array, not adding to queue"; fi
|
if [ "${CONF}" = "${FOUND_CONF}" ]; then FOUND=1; LogText "Found this file already in our configuration files array, not adding to queue"; fi
|
||||||
|
Loading…
x
Reference in New Issue
Block a user