Re: Call for testing: OpenSSH 6.8

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

 



On Tue, 24 Feb 2015, Tim Rice wrote:

> I should have spotted that netdb.h was not included yet
> Perhaps
> .....
> --- defines.h.old	2015-02-23 21:35:19.098756406 -0800
> +++ defines.h	2015-02-24 12:22:24.916077846 -0800
> @@ -108,10 +108,9 @@
>  #ifndef HOST_NAME_MAX
>  # if defined(_POSIX_HOST_NAME_MAX)
>  #  define HOST_NAME_MAX _POSIX_HOST_NAME_MAX
> -# elif defined(MAXHOSTNAMELEN)
> +# elif defined(HAVE_NETDB_H)
> +#  include <netdb.h>

I don't think there is any need for "elif defined(HAVE_NETDB_H)" since
we unconditionally include netdb.h in a few places already.

diff --git defines.h defines.h
index d99ef68..b7dd1d9 100644
--- defines.h
+++ defines.h
@@ -106,6 +106,7 @@ enum
 #endif /* MAXPATHLEN */
 
 #ifndef HOST_NAME_MAX
+# include "netdb.h" /* for MAXHOSTNAMELEN */
 # if defined(_POSIX_HOST_NAME_MAX)
 #  define HOST_NAME_MAX _POSIX_HOST_NAME_MAX
 # elif defined(MAXHOSTNAMELEN)
diff --git includes.h includes.h
index c3034e3..2893a54 100644
--- includes.h
+++ includes.h
@@ -27,7 +27,7 @@
 #include <sys/socket.h> /* For CMSG_* */
 
 #ifdef HAVE_LIMITS_H
-# include <limits.h> /* For PATH_MAX */
+# include <limits.h> /* For PATH_MAX, _POSIX_HOST_NAME_MAX */
 #endif
 #ifdef HAVE_BSTRING_H
 # include <bstring.h>
_______________________________________________
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