fix shellcheck warnings

make_release

@@ -10,7 +10,7 @@ tag=`awk '/IPERF_VERSION / {
   print $3 }' src/version.h`
 fi
 
-dirname=`echo $tag $proj | awk '{
+dirname=`echo "$tag $proj" | awk '{
   gsub(/-ALPHA/, "a", $1);
   gsub(/-BETA/, "b", $1);
   gsub(/-RELEASE/, "", $1);
@@ -21,14 +21,14 @@ dirname=`echo $tag $proj | awk '{
 
 do_tag ()
 {
-    git tag -s -m "tagging $tag" $tag
+    git tag -s -m "tagging $tag" "$tag"
 }
 
 do_tar ()
 {
     tarball=${dirname}.tar.gz
-    rm -f ${tarball}
-    git archive --format=tar --prefix ${dirname}/ ${tag} | gzip -9 > ${tarball}
+    rm -f "${tarball}"
+    git archive --format=tar --prefix "${dirname}/" "${tag}" | gzip -9 > "${tarball}"
 
     # Compute SHA256 hash
     case `uname -s` in
@@ -37,7 +37,7 @@ do_tar ()
 	Darwin) sha="shasum -a 256" ;;
 	*) sha=echo ;;
     esac
-    ${sha} ${tarball} | tee ${tarball}.sha256
+    ${sha} "${tarball}" | tee "${tarball}.sha256"
 }
 
 usage ()

test_commands.sh

@@ -8,7 +8,7 @@
 
 if [ $# -ne 1 ]
 then
-  echo "Usage: `basename $0` hostname"
+  echo "Usage: `basename "$0"` hostname"
   exit $E_BADARGS
 fi
 
@@ -17,59 +17,59 @@ set -x
 host=$1
 
 # basic testing
-./src/iperf3 -c $host -V -t 5 -T "test1"
-./src/iperf3 -c $host -u -V -t 5
+./src/iperf3 -c "$host" -V -t 5 -T "test1"
+./src/iperf3 -c "$host" -u -V -t 5
 # omit mode
-./src/iperf3 -c $host -i .3 -O 2 -t 5
+./src/iperf3 -c "$host" -i .3 -O 2 -t 5
 # JSON mode
-./src/iperf3 -c $host -i 1 -J -t 5
+./src/iperf3 -c "$host" -i 1 -J -t 5
 # force V4
-./src/iperf3 -c $host -4 -t 5
-./src/iperf3 -c $host -4 -u -t 5
+./src/iperf3 -c "$host" -4 -t 5
+./src/iperf3 -c "$host" -4 -u -t 5
 # force V6
-./src/iperf3 -c $host -6 -t 5
-./src/iperf3 -c $host -6 -u -t 5
+./src/iperf3 -c "$host" -6 -t 5
+./src/iperf3 -c "$host" -6 -u -t 5
 # FQ rate
-./src/iperf3 -c $host -V -t 5 --fq-rate 5m
-./src/iperf3 -c $host -u -V -t 5 --fq-rate 5m
+./src/iperf3 -c "$host" -V -t 5 --fq-rate 5m
+./src/iperf3 -c "$host" -u -V -t 5 --fq-rate 5m
 # SCTP
-./src/iperf3 -c $host --sctp -V -t 5
+./src/iperf3 -c "$host" --sctp -V -t 5
 # parallel streams
-./src/iperf3 -c $host -P 3 -t 5 
-./src/iperf3 -c $host -u -P 3 -t 5
+./src/iperf3 -c "$host" -P 3 -t 5 
+./src/iperf3 -c "$host" -u -P 3 -t 5
 # reverse mode
-./src/iperf3 -c $host -P 2 -t 5 -R
-./src/iperf3 -c $host -u -P 2 -t 5 -R
+./src/iperf3 -c "$host" -P 2 -t 5 -R
+./src/iperf3 -c "$host" -u -P 2 -t 5 -R
 # bidirectional mode
-./src/iperf3 -c $host -P 2 -t 5 --bidir
-./src/iperf3 -c $host -u -P 2 -t 5 --bidir
+./src/iperf3 -c "$host" -P 2 -t 5 --bidir
+./src/iperf3 -c "$host" -u -P 2 -t 5 --bidir
 # zero copy
-./src/iperf3 -c $host -Z -t 5 
-./src/iperf3 -c $host -Z -t 5 -R
+./src/iperf3 -c "$host" -Z -t 5 
+./src/iperf3 -c "$host" -Z -t 5 -R
 # window size
-./src/iperf3 -c $host -t 5 -w 8M 
+./src/iperf3 -c "$host" -t 5 -w 8M 
 # -n flag
-./src/iperf3 -c $host -n 5M  
-./src/iperf3 -c $host -n 5M -u -b1G
+./src/iperf3 -c "$host" -n 5M  
+./src/iperf3 -c "$host" -n 5M -u -b1G
 # -n flag with -R
-./src/iperf3 -c $host -n 5M -R
-./src/iperf3 -c $host -n 5M -u -b1G -R
+./src/iperf3 -c "$host" -n 5M -R
+./src/iperf3 -c "$host" -n 5M -u -b1G -R
 # conflicting -n -t flags
-./src/iperf3 -c $host -n 5M -t 5
+./src/iperf3 -c "$host" -n 5M -t 5
 # -k mode
-./src/iperf3 -c $host -k 1K  
-./src/iperf3 -c $host -k 1K -u -b1G
+./src/iperf3 -c "$host" -k 1K  
+./src/iperf3 -c "$host" -k 1K -u -b1G
 # -k mode with -R
-./src/iperf3 -c $host -k 1K -R
-./src/iperf3 -c $host -k 1K -u -b1G -R
+./src/iperf3 -c "$host" -k 1K -R
+./src/iperf3 -c "$host" -k 1K -u -b1G -R
 # CPU affinity
-./src/iperf3 -c $host -A 2/2
-./src/iperf3 -c $host -A 2/2 -u -b1G
+./src/iperf3 -c "$host" -A 2/2
+./src/iperf3 -c "$host" -A 2/2 -u -b1G
 # Burst mode
-./src/iperf3 -c $host -u -b1G/100
+./src/iperf3 -c "$host" -u -b1G/100
 # change MSS
-./src/iperf3 -c $host -M 1000 -V
+./src/iperf3 -c "$host" -M 1000 -V
 # test congestion control option (linux only)
-./src/iperf3 -c $host -C reno -V
+./src/iperf3 -c "$host" -C reno -V