[openssh-commits] [openssh] branch master updated: upstream: Don't check compressions stats when ssh does not support

[git+noreply at mindrot.org]

This is an automated email from the git hooks/post-receive script.

dtucker pushed a commit to branch master
in repository openssh.

The following commit(s) were added to refs/heads/master by this push:
     new c99a30d30 upstream: Don't check compressions stats when ssh does not support
c99a30d30 is described below

commit c99a30d30a5d2af6fec30b9b0d85aa9b252760c9
Author: dtucker at openbsd.org <dtucker at openbsd.org>
AuthorDate: Sat Dec 6 03:23:27 2025 +0000

    upstream: Don't check compressions stats when ssh does not support
    
    compression.
    
    OpenBSD-Regress-ID: 026db51b2654a949e9a10b908443dab83b64c74a
---
 regress/multiplex.sh | 28 ++++++++++++++++++----------
 1 file changed, 18 insertions(+), 10 deletions(-)

diff --git a/regress/multiplex.sh b/regress/multiplex.sh
index 6b58387f3..f68af935e 100644
--- a/regress/multiplex.sh
+++ b/regress/multiplex.sh
@@ -1,4 +1,4 @@
-#	$OpenBSD: multiplex.sh,v 1.39 2025/12/05 08:09:34 dtucker Exp $
+#	$OpenBSD: multiplex.sh,v 1.40 2025/12/06 03:23:27 dtucker Exp $
 #	Placed in the Public Domain.
 
 tid="connection multiplexing"
@@ -198,31 +198,39 @@ kill -0 $SSH_PID >/dev/null 2>&1 && fail "exit command failed"
 
 # Enable compression and alternative kex for next conninfo test.
 if $SSH -Q compression | grep zlib at openssh.com >/dev/null; then
-       echo compression yes >>$OBJ/ssh_config
-       echo kexalgorithms curve25519-sha256 >>$OBJ/ssh_config
-       echo ciphers aes128-ctr >>$OBJ/ssh_config
+	compression=yes
+else
+	compression=no
 fi
+echo compression $compression >>$OBJ/ssh_config
+echo kexalgorithms curve25519-sha256 >>$OBJ/ssh_config
+echo ciphers aes128-ctr >>$OBJ/ssh_config
 
 # Restart master and test -O stop command with master using -N
 verbose "test $tid: cmd stop"
 trace "restart master, fork to background"
 start_mux_master
 
-verbose "test $tid: cmd conninfo compression"
+verbose "test $tid: cmd conninfo algos"
 conninfo=`${SSH} -F $OBJ/ssh_config -S $CTL -Oconninfo otherhost` \
      || fail "request remote forward failed"
-if ! echo "$conninfo" | grep "compression zlib" >/dev/null ||
-    ! echo "$conninfo" | grep "compressed" >/dev/null ||
-    ! echo "$conninfo" | grep "kexalgorithm curve25519-sha256" >/dev/null ||
+if ! echo "$conninfo" | grep "kexalgorithm curve25519-sha256" >/dev/null ||
     ! echo "$conninfo" | grep "cipher aes128-ctr" >/dev/null; then
-       fail "conninfo compression"
+	fail "conninfo algos"
+fi
+if [ "$compression" = "yes" ]; then
+	verbose "test $tid: cmd conninfo compression"
+	if ! echo "$conninfo" | grep "compression zlib" >/dev/null ||
+	    ! echo "$conninfo" | grep "compressed" >/dev/null; then
+		fail "conninfo compression"
+	fi
 fi
 
 # start a long-running command then immediately request a stop
 ${SSH} -F $OBJ/ssh_config -S $CTL otherhost "sleep 10; exit 0" \
      >>$TEST_REGRESS_LOGFILE 2>&1 &
 SLEEP_PID=$!
-${SSH} -F $OBJ/ssh_config -S $CTL -Ostop otherhost >>$TEST_REGRESS_LOGFILE 2>&1 \
+${SSH} -F$OBJ/ssh_config -S$CTL -Ostop otherhost >>$TEST_REGRESS_LOGFILE 2>&1 \
     || fail "send stop command failed"
 
 # wait until both long-running command and master have exited.

-- 
To stop receiving notification emails like this one, please contact
djm at mindrot.org.