From beae0ce19562dbf9a611c6607445d970a9d5d3ac Mon Sep 17 00:00:00 2001 From: Karsten Weiss Date: Wed, 14 Sep 2016 12:11:51 +0200 Subject: [PATCH] run_{rp,application}_banner(): Fix unassigned variables. This commit fixes the following two instances of referenced but not assigned variables: ``` In testssl.sh line 1159: rp_banners="$rp_bannersline" ^-- SC2154: rp_bannersline is referenced but not assigned. In testssl.sh line 1193: app_banners="$app_bannersline" ^-- SC2154: app_bannersline is referenced but not assigned. ``` Found by ShellCheck. --- testssl.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/testssl.sh b/testssl.sh index 5c9619e..fdf2f21 100755 --- a/testssl.sh +++ b/testssl.sh @@ -1156,7 +1156,7 @@ run_rp_banner() { first=false fi emphasize_stuff_in_headers "$line" - rp_banners="$rp_bannersline" + rp_banners="${rp_banners}${line}" done < $TMPFILE fileout "rp_header" "INFO" "Reverse proxy banner(s) found: $rp_banners" fi @@ -1190,7 +1190,7 @@ run_application_banner() { first=false fi emphasize_stuff_in_headers "$line" - app_banners="$app_bannersline" + app_banners="${app_banners}${line}" done fileout "app_banner" "WARN" "Application Banners found: $app_banners" fi