From ce77738d950ebbbee73d2ef5fabe5e71d4c871a7 Mon Sep 17 00:00:00 2001 From: Paul Elliott Date: Wed, 20 Oct 2021 15:59:33 +0100 Subject: [PATCH] Remove bash specific code Use case pattern matching instead of multiline split, given there is only the well formatted PIDs to match on this should be safe. Signed-off-by: Paul Elliott --- tests/ssl-opt.sh | 24 ++++++++++-------------- 1 file changed, 10 insertions(+), 14 deletions(-) diff --git a/tests/ssl-opt.sh b/tests/ssl-opt.sh index 2aad39c4f..d49f3f5be 100755 --- a/tests/ssl-opt.sh +++ b/tests/ssl-opt.sh @@ -559,6 +559,8 @@ has_mem_err() { # Wait for process $2 named $3 to be listening on port $1. Print error to $4. if type lsof >/dev/null 2>/dev/null; then wait_app_start() { + newline=' +' START_TIME=$(date +%s) if [ "$DTLS" -eq 1 ]; then proto=UDP @@ -567,20 +569,14 @@ if type lsof >/dev/null 2>/dev/null; then fi # Make a tight loop, server normally takes less than 1s to start. while true; do - SERVER_PIDS=$(lsof -a -n -b -i "$proto:$1" -F p | cut -c2-) - SERVER_FOUND=false - # When proxies are used, more than one PID can be listening on - # the same port. Each PID will be on its own line. - while read -r PID; do - if [[ $PID == $2 ]]; then - SERVER_FOUND=true - break - fi - done <<< "$SERVER_PIDS" - - if ($SERVER_FOUND == true); then - break - fi + SERVER_PIDS=$(lsof -a -n -b -i "$proto:$1" -F p) + # When we use a proxy, it will be listening on the same port we + # are checking for as well as the server and lsof will list both. + # If multiple PIDs are returned, each one will be on a separate + # line, each prepended with 'p'. + case ${newline}${SERVER_PIDS}${newline} in + *${newline}p${2}${newline}*) break;; + esac if [ $(( $(date +%s) - $START_TIME )) -gt $DOG_DELAY ]; then echo "$3 START TIMEOUT" echo "$3 START TIMEOUT" >> $4