diff options
author | rl1987 <rl1987@sdf.lonestar.org> | 2019-01-22 15:14:16 +0200 |
---|---|---|
committer | rl1987 <rl1987@sdf.lonestar.org> | 2019-01-22 15:14:16 +0200 |
commit | 00fff96e4884e295a767532aa570abf8d27e75ee (patch) | |
tree | 2b1e9fb8b344593963f837788c8ba87571c28d2f | |
parent | d1af4d65df8b5ef8049707029335049db147829f (diff) | |
download | tor-00fff96e4884e295a767532aa570abf8d27e75ee.tar.gz tor-00fff96e4884e295a767532aa570abf8d27e75ee.zip |
Fix shellcheck warning in test_rebind.sh
-rw-r--r-- | changes/bug29063 | 2 | ||||
-rwxr-xr-x | src/test/test_rebind.sh | 9 |
2 files changed, 9 insertions, 2 deletions
diff --git a/changes/bug29063 b/changes/bug29063 new file mode 100644 index 0000000000..8cbcbebc6e --- /dev/null +++ b/changes/bug29063 @@ -0,0 +1,2 @@ + o Code simplification and refactoring (shell scripts): + - Fix issues shellcheck found in test_rebind.sh. Resolves issue 29063. diff --git a/src/test/test_rebind.sh b/src/test/test_rebind.sh index 498072de35..a8f07c7c1e 100755 --- a/src/test/test_rebind.sh +++ b/src/test/test_rebind.sh @@ -15,10 +15,15 @@ fi exitcode=0 tmpdir= -clean () { test -n "$tmpdir" && test -d "$tmpdir" && rm -rf "$tmpdir" || :; } +clean () { + if [ -n "$tmpdir" ] && [ -d "$tmpdir" ]; then + rm -rf "$tmpdir" + fi +} + trap clean EXIT HUP INT TERM -tmpdir="`mktemp -d -t tor_rebind_test.XXXXXX`" +tmpdir="$(mktemp -d -t tor_rebind_test.XXXXXX)" if [ -z "$tmpdir" ]; then echo >&2 mktemp failed exit 2 |