better matching of empty server banners

.. which could have led to borken JSON too
This commit is contained in:
Dirk 2018-03-01 16:01:05 +01:00
parent 8c9fdc6cc9
commit b1b30b3fb8

View File

@ -2161,7 +2161,7 @@ run_server_banner() {
grep -ai '^Server' $HEADERFILE >$TMPFILE
if [[ $? -eq 0 ]]; then
serverbanner=$(sed -e 's/^Server: //' -e 's/^server: //' $TMPFILE)
if [[ x"$serverbanner" == "x\n" ]] || [[ x"$serverbanner" == "x\n\r" ]] || [[ -z "$serverbanner" ]]; then
if [[ "$serverbanner" == $'\n' ]] || [[ "$serverbanner" == $'\r' ]] || [[ "$serverbanner" == $'\n\r' ]] || [[ -z "$serverbanner" ]]; then
outln "banner exists but empty string"
fileout "$jsonID" "INFO" "Server banner is empty"
else