From c51ab0a8502e9387f3bdecb53699b422464fd5b1 Mon Sep 17 00:00:00 2001 From: Nikolay Borisov Date: Thu, 25 May 2017 12:08:46 +0300 Subject: [PATCH] generic/108: Fix return value check from _get_scsi_debug_dev _get_scsi_debug_dev is supposed to return a "/dev/$device". However, in case the scsi device is not mapped to a disk, hence /dev/sd* doesn't exist, then get_scsi_debug_dev would return only the "/dev/" string. In generic/108 we check whether return value is "" and only then consider it a failure. This behavior allows the test to erroneously consider _get_scsi_debug_dev succeeded even if it returned a malformed string. Fix this by correctly checking whether the return value is "/dev/" Signed-off-by: Nikolay Borisov Reviewed-by: Eryu Guan Signed-off-by: Eryu Guan --- tests/generic/108 | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/generic/108 b/tests/generic/108 index cc8d7038..1525dfb6 100755 --- a/tests/generic/108 +++ b/tests/generic/108 @@ -64,7 +64,7 @@ rm -f $seqres.full # _get_scsi_debug_dev returns a scsi debug device with 128M in size by default SCSI_DEBUG_DEV=`_get_scsi_debug_dev` -if [ "$SCSI_DEBUG_DEV" == "" ]; then +if [ "$SCSI_DEBUG_DEV" == "/dev/" ]; then _fail "Failed to initialize scsi debug device" fi echo "SCSI debug device $SCSI_DEBUG_DEV" >>$seqres.full -- 2.39.5