test if FSSTRESS_PROG is a null string dosen't make sense because it has just been set. Here fix it by testing if it is an executable file. Signed-off-by: Wang Shilong <wangshilong1991@xxxxxxxxx> --- common/config | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/common/config b/common/config index d68d4d0..64506a2 100644 --- a/common/config +++ b/common/config @@ -135,7 +135,7 @@ export UMOUNT_PROG="`set_prog_path umount`" [ "$UMOUNT_PROG" = "" ] && _fatal "umount not found" export FSSTRESS_PROG="./ltp/fsstress" -[ "$FSSTRESS_PROG" = "" ] && _fatal "fsstress not found" +[ ! -x $FSSTRESS_PROG ] && _fatal "fsstress not found or executable" export PERL_PROG="`set_prog_path perl`" [ "$PERL_PROG" = "" ] && _fatal "perl not found" -- 1.8.3.1 -- To unsubscribe from this list: send the line "unsubscribe fstests" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
