From 7a5ebe41f2a8d7a64b3fce31ec48eb38428bc3d7 Mon Sep 17 00:00:00 2001 From: Marius Strobl Date: Sun, 28 Oct 2007 15:41:11 +0000 Subject: [PATCH] - Given that we tell the compiler that struct ip is packed and 32-bit aligned, GCC 4.2.1 also generates code for sendudp() that assumes this alignment. GCC 4.2.1 however doesn't 32-bit align wbuf, causing the loader to crash due to an unaligned access of wbuf in sendudp() when netbooting sparc64. Solve this by specifying wbuf as packed and 32-bit aligned, too. As for lastdata and readudp() this currently is no issue when compiled with GCC 4.2.1, though give lastdata the same treatment as wbuf for consistency and possibility of being affected in the future. - Sprinkle const on a lookup table. Approved by: re (kensmith) --- lib/libstand/tftp.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/lib/libstand/tftp.c b/lib/libstand/tftp.c index 0cbfa520a4ff..fd1b4ee7a921 100644 --- a/lib/libstand/tftp.c +++ b/lib/libstand/tftp.c @@ -95,10 +95,10 @@ struct tftp_handle { u_char header[HEADER_SIZE]; struct tftphdr t; u_char space[RSPACE]; - } lastdata; + } __packed __aligned(4) lastdata; }; -static int tftperrors[8] = { +static const int tftperrors[8] = { 0, /* ??? */ ENOENT, EPERM, @@ -175,7 +175,7 @@ tftp_makereq(h) u_char header[HEADER_SIZE]; struct tftphdr t; u_char space[FNAME_SIZE + 6]; - } wbuf; + } __packed __aligned(4) wbuf; char *wtail; int l; ssize_t res; @@ -218,7 +218,7 @@ tftp_getnextblock(h) struct { u_char header[HEADER_SIZE]; struct tftphdr t; - } wbuf; + } __packed __aligned(4) wbuf; char *wtail; int res; struct tftphdr *t;