From f66c6d614efdf7ef2641632f160ffe9de460b565 Mon Sep 17 00:00:00 2001 From: Paul Stewart Date: Fri, 22 Oct 2010 10:31:27 -0700 Subject: [PATCH] Complain (and more importantly don't blindly run) if AP or Client IP is unset Add small if statement in cros_run_wifi_tests to balk if we didn't get valid cell info from the server. BUG=none TEST=reboot everything and rerun while testbed server hasn't caught up yet Review URL: http://codereview.chromium.org/4069001 --- bin/cros_run_wifi_tests.sh | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/bin/cros_run_wifi_tests.sh b/bin/cros_run_wifi_tests.sh index f948a4ef4c..babb3d1b02 100755 --- a/bin/cros_run_wifi_tests.sh +++ b/bin/cros_run_wifi_tests.sh @@ -123,6 +123,10 @@ while read line; do set $line if [ "${FLAGS_cell}" = "$cell_no" -o "${FLAGS_client}" = "$1" -o \ "${FLAGS_client}" = "$4" ] ; then + if [ "$5" = "0.0.0.0" -o "$4" = "0.0.0.0" ]; then + # Error -- these should never be zeroes + break + fi echo "$4" echo "router_addr=$5" if [ "$6" != "0.0.0.0" ] ; then