From 369f9a5c41e35f63787c2c0f077df5f44048c87f Mon Sep 17 00:00:00 2001
From: Minh Diep <mdiep@whamcloud.com>
Date: Thu, 22 Mar 2012 18:06:06 -0700
Subject: [PATCH] LU-1254 iokit: should not use single '[]'

Single [] checking for empty variable is dangerous.
When the variable is undefined, it will fall through
without double quote

Signed-off-by: Minh Diep <mdiep@whamcloud.com>
Change-Id: Iec49cd704be8d106cce07032dc20f060f6044e8a
Reviewed-on: http://review.whamcloud.com/2373
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Wei Liu <sarah@whamcloud.com>
Reviewed-by: Yu Jian <yujian@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
---
 lustre-iokit/obdfilter-survey/libecho | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lustre-iokit/obdfilter-survey/libecho b/lustre-iokit/obdfilter-survey/libecho
index c34d9c8..9a4e8c4 100644
--- a/lustre-iokit/obdfilter-survey/libecho
+++ b/lustre-iokit/obdfilter-survey/libecho
@@ -217,7 +217,7 @@ cleanup () {
     shift
     for ((i = 0; i < $ndevs; i++)); do
         host=${host_names[$i]}
-        if [ -n ${do_teardown_ec[$i]} ]; then
+        if [[ -n "${do_teardown_ec[$i]}" ]]; then
             teardown_ec_devno $host ${client_names[$i]}
         fi
     done
-- 
1.8.3.1