diff options
author | Todd C. Miller <millert@cvs.openbsd.org> | 2017-10-25 20:08:37 +0000 |
---|---|---|
committer | Todd C. Miller <millert@cvs.openbsd.org> | 2017-10-25 20:08:37 +0000 |
commit | 85c1fe8dce3181720f253000d16c81765c8e8720 (patch) | |
tree | 96a4d56cb27e77866ddad18949a6bf50cc67fd68 | |
parent | df3b012ad9d35c6bbf33f8de00caa1e9ed6ad525 (diff) |
Use printenv to test whether an SSH_USER_AUTH is set instead of
using $SSH_USER_AUTH. The latter won't work with csh which treats
unknown variables as an error when expanding them. OK markus@
-rw-r--r-- | regress/usr.bin/ssh/authinfo.sh | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/regress/usr.bin/ssh/authinfo.sh b/regress/usr.bin/ssh/authinfo.sh index e725296c90a..9bd0a4d8366 100644 --- a/regress/usr.bin/ssh/authinfo.sh +++ b/regress/usr.bin/ssh/authinfo.sh @@ -1,4 +1,4 @@ -# $OpenBSD: authinfo.sh,v 1.1 2017/06/24 06:35:24 djm Exp $ +# $OpenBSD: authinfo.sh,v 1.2 2017/10/25 20:08:36 millert Exp $ # Placed in the Public Domain. tid="authinfo" @@ -6,7 +6,7 @@ tid="authinfo" # Ensure the environment variable doesn't leak when ExposeAuthInfo=no. verbose "ExposeAuthInfo=no" env SSH_USER_AUTH=blah ${SSH} -F $OBJ/ssh_proxy x \ - 'test -z "$SSH_USER_AUTH"' || fail "SSH_USER_AUTH present" + 'printenv SSH_USER_AUTH >/dev/null' && fail "SSH_USER_AUTH present" verbose "ExposeAuthInfo=yes" echo ExposeAuthInfo=yes >> $OBJ/sshd_proxy |