diff options
author | ngie <ngie@FreeBSD.org> | 2014-11-15 05:16:16 +0000 |
---|---|---|
committer | ngie <ngie@FreeBSD.org> | 2014-11-15 05:16:16 +0000 |
commit | c1b4b35c7e4bad970b85bb047fc9e724499f6248 (patch) | |
tree | b4001d2142769ea8914adb439c76c40b1bdf7ca1 /tools | |
parent | 20a08c7aa6060128395fef845ebf54389be078df (diff) | |
download | FreeBSD-src-c1b4b35c7e4bad970b85bb047fc9e724499f6248.zip FreeBSD-src-c1b4b35c7e4bad970b85bb047fc9e724499f6248.tar.gz |
MFC r273627,r273628:
r273627:
- Print out "Bail out!" in die(..) so prove terminates immediately
- Handle the output from newer versions of openssl md5, similar to what
pjd@ did in r248304
Sponsored by: EMC / Isilon Storage Division
r273628:
Move the redirection to stderr out of the cmd variable assignment
Putting 2>/dev/null in cmd= escapes the redirection operation, which causes
mdconfig to think it's a filename
Sponsored by: EMC / Isilon Storage Division
Diffstat (limited to 'tools')
-rwxr-xr-x | tools/regression/zfs/misc.sh | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/tools/regression/zfs/misc.sh b/tools/regression/zfs/misc.sh index d96aa22..b52481e 100755 --- a/tools/regression/zfs/misc.sh +++ b/tools/regression/zfs/misc.sh @@ -39,6 +39,7 @@ fi die() { echo "${1}" > /dev/stderr + echo "Bail out!" exit 1 } @@ -166,8 +167,8 @@ create_memdisk() if [ -n "${devname}" ]; then devparam="-u ${devname}" fi - cmd="mdconfig -a -t swap -s ${size} ${devparam} 2>/dev/null" - DISKNAME=`${cmd}` || die "failed: ${cmd}" + cmd="mdconfig -a -t swap -s ${size} ${devparam}" + DISKNAME=`$cmd 2>/dev/null` || die "failed: ${cmd}" if [ -n "${devname}" ]; then DISKNAME="${devname}" fi @@ -363,7 +364,7 @@ files_destroy() name_create() { - echo "zfstest_`dd if=/dev/urandom bs=1k count=1 2>/dev/null | openssl md5 | cut -b -8`" + echo "zfstest_`dd if=/dev/urandom bs=1k count=1 2>/dev/null | openssl md5 | awk '{ print $NF }'`" } names_create() |