diff options
author | Darren Tucker <dtucker@dtucker.net> | 2018-03-22 17:00:28 +1100 |
---|---|---|
committer | Darren Tucker <dtucker@dtucker.net> | 2018-03-22 17:00:28 +1100 |
commit | c6cb2565c9285eb54fa9dfbb3890f5464aff410f (patch) | |
tree | 7836f40c3a7ee5b3d148778e2ecace572c24d5d3 | |
parent | 4c4e7f783b43b264c247233acb887ee10ed4ce4d (diff) |
Save $? before case statement.
In some shells (FreeBSD 9, ash) the case statement resets $?, so save
for later testing.
-rw-r--r-- | regress/key-options.sh | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/regress/key-options.sh b/regress/key-options.sh index e03cae585..d680737c1 100644 --- a/regress/key-options.sh +++ b/regress/key-options.sh | |||
@@ -101,9 +101,10 @@ check_valid_before() { | |||
101 | sed "s/.*/$opts &/" $origkeys >$authkeys | 101 | sed "s/.*/$opts &/" $origkeys >$authkeys |
102 | verbose "key option expiry-time $which" | 102 | verbose "key option expiry-time $which" |
103 | ${SSH} -q -F $OBJ/ssh_proxy somehost true | 103 | ${SSH} -q -F $OBJ/ssh_proxy somehost true |
104 | r=$? | ||
104 | case "$expect" in | 105 | case "$expect" in |
105 | fail) test $? -eq 0 && fail "key option succeeded $which" ;; | 106 | fail) test $r -eq 0 && fail "key option succeeded $which" ;; |
106 | pass) test $? -ne 0 && fail "key option failed $which" ;; | 107 | pass) test $r -ne 0 && fail "key option failed $which" ;; |
107 | *) fatal "unknown expectation $expect" ;; | 108 | *) fatal "unknown expectation $expect" ;; |
108 | esac | 109 | esac |
109 | } | 110 | } |