Re: Call for testing: OpenSSH 6.9

[Date Prev] [Date Next] [Thread Prev] [Thread Next] [Date Index] [Thread Index]

 



On Tue, 30 Jun 2015, Damien Miller wrote:

| On Mon, 29 Jun 2015, Tim Rice wrote:
| 
| > On Tue, 30 Jun 2015, Damien Miller wrote:
| > 
| > | I think we should just disable the test if the host doesn't support IPv6.
| > | 
| > | diff --git a/regress/cfgparse.sh b/regress/cfgparse.sh
| > | index 7f377d8..e19b4d0 100644
| > | --- a/regress/cfgparse.sh
| > | +++ b/regress/cfgparse.sh
| > | @@ -3,6 +3,12 @@
| > |  
| > |  tid="config parse"
| > |  
| > | +# Possessing struct addrinfo is a reasonable proxy for IPv6 support.
| > | +if ! config_defined HAVE_STRUCT_ADDRINFO ; then
| > 
| > Did you mean HAVE_STRUCT_IN6_ADDR ?
| 
| That's even betterer. Ok?
| 
| diff --git a/regress/cfgparse.sh b/regress/cfgparse.sh
| index 7f377d8..a5296a2 100644
| --- a/regress/cfgparse.sh
| +++ b/regress/cfgparse.sh
| @@ -3,6 +3,12 @@
|  
|  tid="config parse"
|  
| +# This is a reasonable proxy for IPv6 support.
| +if ! config_defined HAVE_STRUCT_IN6_ADDR ; then
| +	echo "skipped (not supported on this platform)"
| +	exit 0
| +fi
| +
|  # We need to use the keys generated for the regression test because sshd -T
|  # will fail if we're not running with SUDO (no permissions for real keys) or
|  # if we are # running tests on a system that has never had sshd installed
| _______________________________________________

Seems a shame to disable the whole test.
Is this too ugly?
............
--- cfgparse.sh.old	2015-06-01 22:43:48.721636775 -0700
+++ cfgparse.sh	2015-06-29 21:33:04.793724856 -0700
@@ -3,6 +3,11 @@
 
 tid="config parse"
 
+# This is a reasonable proxy for IPv6 support.
+if ! config_defined HAVE_STRUCT_IN6_ADDR ; then
+	SKIP_IPV6=yes
+fi
+
 # We need to use the keys generated for the regression test because sshd -T
 # will fail if we're not running with SUDO (no permissions for real keys) or
 # if we are # running tests on a system that has never had sshd installed
@@ -26,6 +31,9 @@
 cat > $OBJ/sshd_config.0 <<EOD
 listenaddress 1.2.3.4:1234
 listenaddress 1.2.3.4:5678
+EOD
+
+[ X${SKIP_IPV6} = Xyes ]  ||  cat > $OBJ/sshd_config.0 <<EOD
 listenaddress [::1]:1234
 listenaddress [::1]:5678
 EOD
@@ -37,6 +45,9 @@
 port 1234
 port 5678
 listenaddress 1.2.3.4
+EOD
+
+[ X${SKIP_IPV6} = Xyes ]  ||  cat > $OBJ/sshd_config.1 <<EOD
 listenaddress ::1
 EOD
 ($SUDO ${SSHD} -T -f $OBJ/sshd_config.1 | \
@@ -47,11 +58,14 @@
 cat > $OBJ/sshd_config.1 <<EOD
 ${SSHD_KEYS}
 listenaddress 1.2.3.4
-listenaddress ::1
 port 1234
 port 5678
 addressfamily any
 EOD
+
+[ X${SKIP_IPV6} = Xyes ]  ||  cat > $OBJ/sshd_config.1 <<EOD
+listenaddress ::1
+EOD
 ($SUDO ${SSHD} -T -f $OBJ/sshd_config.1 | \
  grep 'listenaddress ' >$OBJ/sshd_config.2 &&
  diff $OBJ/sshd_config.0 $OBJ/sshd_config.2) || \
............

-- 
Tim Rice				Multitalents
tim@xxxxxxxxxxxxxxxx


_______________________________________________
openssh-unix-dev mailing list
openssh-unix-dev@xxxxxxxxxxx
https://lists.mindrot.org/mailman/listinfo/openssh-unix-dev



[Date Prev] [Date Next] [Thread Prev] [Thread Next] [Date Index] [Thread Index]

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux