From: Daniel Stenberg <daniel@haxx.se>
Date: Fri, 4 Jan 2002 09:17:52 +0000 (+0000)
Subject: The buffer in ftp_pasv_verbose(), used for gethostbyaddr_r(), is now defined
X-Git-Tag: curl-7_9_3-pre1~27
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=17a8bf212f26ba17d14a6e00048ab6f15b4baf89;p=thirdparty%2Fcurl.git

The buffer in ftp_pasv_verbose(), used for gethostbyaddr_r(), is now defined
to become properly 8-byte aligned on 64-bit archs. Philip Gladstone reported.
---

diff --git a/lib/ftp.c b/lib/ftp.c
index 56449f6a4b..0ac5d4d2b7 100644
--- a/lib/ftp.c
+++ b/lib/ftp.c
@@ -861,13 +861,17 @@ ftp_pasv_verbose(struct connectdata *conn,
 #ifdef HAVE_INET_NTOA_R
   char ntoa_buf[64];
 #endif
-  char hostent_buf[9000];
+  /* The array size trick below is to make this a large chunk of memory
+     suitably 8-byte aligned on 64-bit platforms. This was thoughtfully
+     suggested by Philip Gladstone. */
+  long bigbuf[9000 / sizeof(long)];
 
 #if defined(HAVE_INET_ADDR)
   in_addr_t address;
 # if defined(HAVE_GETHOSTBYADDR_R)
   int h_errnop;
 # endif
+  char *hostent_buf = (char *)bigbuf; /* get a char * to the buffer */
 
   address = inet_addr(newhost);
 # ifdef HAVE_GETHOSTBYADDR_R