Call for testing: OpenSSH 7.3

Darren Tucker dtucker at
Fri Jul 22 21:37:32 AEST 2016

On Fri, Jul 22, 2016 at 12:05:53PM +0200, Corinna Vinschen wrote:
> This version doesn't build on Cygwin anymore.  The reason is that
> various configure tests fail.
> The culprit is the new definition of IPPORT_RESERVED to 0 in


How about putting it in defines.h instead?  includes.h includes
netinet/in.h from whence the definition of IPPORT_RESERVED is, on Cygwin
at least, seems to be protected against multiple inclusion.  Putting it
there means only one definition in a file that we don't sync with OpenBSD.

diff --git a/ b/
index 21ef389..2cd6a6f 100644
--- a/
+++ b/
@@ -589,8 +589,9 @@ case "$host" in
 		[Define if you want to disable shadow passwords])
 		[Define if X11 doesn't support AF_UNIX sockets on that system])
-		[Cygwin has no notion of ports only accessible to superusers])
+		[Define if the concept of ports only accessible to
+		superusers isn't known])
 		[Define if your platform needs to skip post auth
 		file descriptor passing])
diff --git a/defines.h b/defines.h
index a438ddd..c099df6 100644
--- a/defines.h
+++ b/defines.h
@@ -43,6 +43,17 @@ enum
+ * Cygwin doesn't really have a notion of reserved ports but for backward
+ * compatibility they define it to 1024 in netinet/in.h inside an enum.  We
+ * don't actually want that restriction so we want to set that to zero, but
+ * we can't do it direct in config.h because it'll cause a conflicting
+ * definition the first time we include netinet/in.h.
+ */
  * Definitions for IP type of service (ip_tos)
 #include <netinet/in_systm.h>

Darren Tucker (dtucker at
GPG key 11EAA6FA / A86E 3E07 5B19 5880 E860  37F4 9357 ECEF 11EA A6FA (new)
    Good judgement comes with experience. Unfortunately, the experience
usually comes from bad judgement.

More information about the openssh-unix-dev mailing list