From: Nikolay Borisov Date: Thu, 25 May 2017 09:08:46 +0000 (+0300) Subject: generic/108: Fix return value check from _get_scsi_debug_dev X-Git-Tag: v2022.05.01~2032 X-Git-Url: https://www.infradead.org/git/?a=commitdiff_plain;h=c51ab0a8502e9387f3bdecb53699b422464fd5b1;p=users%2Fhch%2Fxfstests-dev.git 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 --- diff --git a/tests/generic/108 b/tests/generic/108 index cc8d70384..1525dfb64 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