--- src/contrib/ntpd/Attic/ntp_msg.c	2004/11/09 21:29:34	1.2
+++ src/contrib/ntpd/Attic/ntp_msg.c	2004/12/14 16:05:57	1.3
@@ -1,4 +1,4 @@
-/*	$OpenBSD: src/usr.sbin/ntpd/ntp_msg.c,v 1.11 2004/10/22 21:24:20 henning Exp $ */
+/*	$OpenBSD: src/usr.sbin/ntpd/ntp_msg.c,v 1.14 2004/12/14 06:27:13 dtucker Exp $ */
 
 /*
  * Copyright (c) 2003, 2004 Henning Brauer <henning@openbsd.org>
@@ -34,40 +34,25 @@ ntp_getmsg(char *p, ssize_t len, struct
 		return (-1);
 	}
 
-	memcpy(&msg->status, p, sizeof(msg->status));
-	p += sizeof(msg->status);
-	memcpy(&msg->stratum, p, sizeof(msg->stratum));
-	p += sizeof(msg->stratum);
-	memcpy(&msg->ppoll, p, sizeof(msg->ppoll));
-	p += sizeof(msg->ppoll);
-	memcpy(&msg->precision, p, sizeof(msg->precision));
-	p += sizeof(msg->precision);
-	memcpy(&msg->rootdelay.int_part, p, sizeof(msg->rootdelay.int_part));
-	p += sizeof(msg->rootdelay.int_part);
-	memcpy(&msg->rootdelay.fraction, p, sizeof(msg->rootdelay.fraction));
-	p += sizeof(msg->rootdelay.fraction);
-	memcpy(&msg->dispersion.int_part, p, sizeof(msg->dispersion.int_part));
-	p += sizeof(msg->dispersion.int_part);
-	memcpy(&msg->dispersion.fraction, p, sizeof(msg->dispersion.fraction));
-	p += sizeof(msg->dispersion.fraction);
-	memcpy(&msg->refid, p, sizeof(msg->refid));
-	p += sizeof(msg->refid);
-	memcpy(&msg->reftime.int_part, p, sizeof(msg->reftime.int_part));
-	p += sizeof(msg->reftime.int_part);
-	memcpy(&msg->reftime.fraction, p, sizeof(msg->reftime.fraction));
-	p += sizeof(msg->reftime.fraction);
-	memcpy(&msg->orgtime.int_part, p, sizeof(msg->orgtime.int_part));
-	p += sizeof(msg->orgtime.int_part);
-	memcpy(&msg->orgtime.fraction, p, sizeof(msg->orgtime.fraction));
-	p += sizeof(msg->orgtime.fraction);
-	memcpy(&msg->rectime.int_part, p, sizeof(msg->rectime.int_part));
-	p += sizeof(msg->rectime.int_part);
-	memcpy(&msg->rectime.fraction, p, sizeof(msg->rectime.fraction));
-	p += sizeof(msg->rectime.fraction);
-	memcpy(&msg->xmttime.int_part, p, sizeof(msg->xmttime.int_part));
-	p += sizeof(msg->xmttime.int_part);
-	memcpy(&msg->xmttime.fraction, p, sizeof(msg->xmttime.fraction));
-	p += sizeof(msg->xmttime.fraction);
+#define	copyin(f,p)	memcpy(&(f), (p), sizeof(f)); (p) += sizeof(f)
+
+	copyin(msg->status, p);
+	copyin(msg->stratum, p);
+	copyin(msg->ppoll, p);
+	copyin(msg->precision, p);
+	copyin(msg->rootdelay.int_parts, p);
+	copyin(msg->rootdelay.fractions, p);
+	copyin(msg->dispersion.int_parts, p);
+	copyin(msg->dispersion.fractions, p);
+	copyin(msg->refid, p);
+	copyin(msg->reftime.int_partl, p);
+	copyin(msg->reftime.fractionl, p);
+	copyin(msg->orgtime.int_partl, p);
+	copyin(msg->orgtime.fractionl, p);
+	copyin(msg->rectime.int_partl, p);
+	copyin(msg->rectime.fractionl, p);
+	copyin(msg->xmttime.int_partl, p);
+	copyin(msg->xmttime.fractionl, p);
 
 	return (0);
 }
@@ -77,44 +62,28 @@ ntp_sendmsg(int fd, struct sockaddr *sa,
     int auth)
 {
 	char		 buf[NTP_MSGSIZE];
-	char		*p;
-	u_int8_t	sa_len;
+	char		*p = buf;
+	socklen_t	sa_len;
+
+#define	copyout(p,f)	memcpy((p), &(f), sizeof(f)); p += sizeof(f)
 
-	p = buf;
-	memcpy(p, &msg->status, sizeof(msg->status));
-	p += sizeof(msg->status);
-	memcpy(p, &msg->stratum, sizeof(msg->stratum));
-	p += sizeof(msg->stratum);
-	memcpy(p, &msg->ppoll, sizeof(msg->ppoll));
-	p += sizeof(msg->ppoll);
-	memcpy(p, &msg->precision, sizeof(msg->precision));
-	p += sizeof(msg->precision);
-	memcpy(p, &msg->rootdelay.int_part, sizeof(msg->rootdelay.int_part));
-	p += sizeof(msg->rootdelay.int_part);
-	memcpy(p, &msg->rootdelay.fraction, sizeof(msg->rootdelay.fraction));
-	p += sizeof(msg->rootdelay.fraction);
-	memcpy(p, &msg->dispersion.int_part, sizeof(msg->dispersion.int_part));
-	p += sizeof(msg->dispersion.int_part);
-	memcpy(p, &msg->dispersion.fraction, sizeof(msg->dispersion.fraction));
-	p += sizeof(msg->dispersion.fraction);
-	memcpy(p, &msg->refid, sizeof(msg->refid));
-	p += sizeof(msg->refid);
-	memcpy(p, &msg->reftime.int_part, sizeof(msg->reftime.int_part));
-	p += sizeof(msg->reftime.int_part);
-	memcpy(p, &msg->reftime.fraction, sizeof(msg->reftime.fraction));
-	p += sizeof(msg->reftime.fraction);
-	memcpy(p, &msg->orgtime.int_part, sizeof(msg->orgtime.int_part));
-	p += sizeof(msg->orgtime.int_part);
-	memcpy(p, &msg->orgtime.fraction, sizeof(msg->orgtime.fraction));
-	p += sizeof(msg->orgtime.fraction);
-	memcpy(p, &msg->rectime.int_part, sizeof(msg->rectime.int_part));
-	p += sizeof(msg->rectime.int_part);
-	memcpy(p, &msg->rectime.fraction, sizeof(msg->rectime.fraction));
-	p += sizeof(msg->rectime.fraction);
-	memcpy(p, &msg->xmttime.int_part, sizeof(msg->xmttime.int_part));
-	p += sizeof(msg->xmttime.int_part);
-	memcpy(p, &msg->xmttime.fraction, sizeof(msg->xmttime.fraction));
-	p += sizeof(msg->xmttime.fraction);
+	copyout(p, msg->status);
+	copyout(p, msg->stratum);
+	copyout(p, msg->ppoll);
+	copyout(p, msg->precision);
+	copyout(p, msg->rootdelay.int_parts);
+	copyout(p, msg->rootdelay.fractions);
+	copyout(p, msg->dispersion.int_parts);
+	copyout(p, msg->dispersion.fractions);
+	copyout(p, msg->refid);
+	copyout(p, msg->reftime.int_partl);
+	copyout(p, msg->reftime.fractionl);
+	copyout(p, msg->orgtime.int_partl);
+	copyout(p, msg->orgtime.fractionl);
+	copyout(p, msg->rectime.int_partl);
+	copyout(p, msg->rectime.fractionl);
+	copyout(p, msg->xmttime.int_partl);
+	copyout(p, msg->xmttime.fractionl);
 
 	if (sa != NULL)
 		sa_len = SA_LEN(sa);