we don't have to cast to caddr_t when calling m_copydata anymore.

the first cut of this diff was made with coccinelle using this spatch:

@rule@
type caddr_t;
expression m, off, len, cp;
@@
-m_copydata(m, off, len, (caddr_t)cp)
+m_copydata(m, off, len, cp)

i had fix it's opinionated idea of formatting by hand though, so
i'm not sure it was worth it.

ok deraadt@ bluhm@

1 files changed, 2 insertions, 2 deletions

diff --git a/sys/net/if_pfsync.c b/sys/net/if_pfsync.c
index b6c48a38339..f6088c8bee5 100644
--- a/sys/net/if_pfsync.c
+++ b/sys/net/if_pfsync.c
@@ -1,4 +1,4 @@
-/*	$OpenBSD: if_pfsync.c,v 1.286 2021/02/19 06:14:07 dlg Exp $	*/
+/*	$OpenBSD: if_pfsync.c,v 1.287 2021/02/25 02:48:21 dlg Exp $	*/
 
 /*
  * Copyright (c) 2002 Michael Shalayeff
@@ -792,7 +792,7 @@ pfsync_input(struct mbuf **mp, int *offp, int proto, int af)
 
 	offset += sizeof(*ph);
 	while (offset <= len - sizeof(subh)) {
-		m_copydata(m, offset, sizeof(subh), (caddr_t)&subh);
+		m_copydata(m, offset, sizeof(subh), &subh);
 		offset += sizeof(subh);
 
 		mlen = subh.len << 2;