Make sure puttygen is new enough to successfully run the PuTTY interop

tests, otherwise skip them.

diff --git a/regress/usr.bin/ssh/test-exec.sh b/regress/usr.bin/ssh/test-exec.sh
index e6994f7..e010c1f 100644 (file)
--- a/regress/usr.bin/ssh/test-exec.sh
+++ b/regress/usr.bin/ssh/test-exec.sh
@@ -1,4 +1,4 @@
-#      $OpenBSD: test-exec.sh,v 1.76 2020/04/04 23:04:41 dtucker Exp $
+#      $OpenBSD: test-exec.sh,v 1.77 2021/02/17 03:59:00 dtucker Exp $
 #      Placed in the Public Domain.
 
 USER=`id -un`
@@ -413,10 +413,11 @@ if test -x "$CONCH" ; then
        REGRESS_INTEROP_CONCH=yes
 fi
 
-# If PuTTY is present and we are running a PuTTY test, prepare keys and
-# configuration
+# If PuTTY is present, new enough and we are running a PuTTY test, prepare
+# keys and configuration.
 REGRESS_INTEROP_PUTTY=no
-if test -x "$PUTTYGEN" -a -x "$PLINK" ; then
+if test -x "$PUTTYGEN" -a -x "$PLINK" &&
+    "$PUTTYGEN" --help 2>&1 | grep -- --new-passphrase >/dev/null; then
        REGRESS_INTEROP_PUTTY=yes
 fi
 case "$SCRIPT" in
@@ -429,13 +430,13 @@ if test "$REGRESS_INTEROP_PUTTY" = "yes" ; then
 
        # Add a PuTTY key to authorized_keys
        rm -f ${OBJ}/putty.rsa2
-       if ! puttygen -t rsa -o ${OBJ}/putty.rsa2 \
+       if ! "$PUTTYGEN" -t rsa -o ${OBJ}/putty.rsa2 \
            --random-device=/dev/urandom \
            --new-passphrase /dev/null < /dev/null > /dev/null; then
-               echo "Your installed version of PuTTY is too old to support --new-passphrase; trying without (may require manual interaction) ..." >&2
-               puttygen -t rsa -o ${OBJ}/putty.rsa2 < /dev/null > /dev/null
+               echo "Your installed version of PuTTY is too old to support --new-passphrase, skipping test" >&2
+               exit 1
        fi
-       puttygen -O public-openssh ${OBJ}/putty.rsa2 \
+       "$PUTTYGEN" -O public-openssh ${OBJ}/putty.rsa2 \
            >> $OBJ/authorized_keys_$USER
 
        # Convert rsa2 host key to PuTTY format
@@ -456,8 +457,6 @@ if test "$REGRESS_INTEROP_PUTTY" = "yes" ; then
        echo "ProxyMethod=5" >> ${OBJ}/.putty/sessions/localhost_proxy
        echo "ProxyTelnetCommand=sh ${SRC}/sshd-log-wrapper.sh ${TEST_SSHD_LOGFILE} ${SSHD} -i -f $OBJ/sshd_proxy" >> ${OBJ}/.putty/sessions/localhost_proxy
        echo "ProxyLocalhost=1" >> ${OBJ}/.putty/sessions/localhost_proxy
-
-       REGRESS_INTEROP_PUTTY=yes
 fi
 
 # create a proxy version of the client config