diff options
author | bz <bz@FreeBSD.org> | 2006-03-30 18:57:04 +0000 |
---|---|---|
committer | bz <bz@FreeBSD.org> | 2006-03-30 18:57:04 +0000 |
commit | 17b93d79f6279c53f5b86032a17d2eddacc1fc2d (patch) | |
tree | 0672aa33c1c4aaecb43005db71ffdfea5c87f440 /sys/netipsec/xform.h | |
parent | f0667561aafda6d724304328a9441ef3d978e30e (diff) | |
download | FreeBSD-src-17b93d79f6279c53f5b86032a17d2eddacc1fc2d.zip FreeBSD-src-17b93d79f6279c53f5b86032a17d2eddacc1fc2d.tar.gz |
Fix more stack corruptions on amd64.
Vararg functions have a different calling convention than regular
functions on amd64. Casting a varag function to a regular one to
match the function pointer declaration will hide the varargs from
the caller and we will end up with an incorrectly setup stack.
Entirely remove the varargs from these functions and change the
functions to match the declaration of the function pointers.
Remove the now unnecessary casts.
Also change static struct ipprotosw[] to two independent
protosw/ip6protosw definitions to remove an unnecessary cast.
PR: amd64/95008
Submitted and tested by: Mats Palmgren
Reviewed by: rwatson
MFC after: 3 days
Diffstat (limited to 'sys/netipsec/xform.h')
-rw-r--r-- | sys/netipsec/xform.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sys/netipsec/xform.h b/sys/netipsec/xform.h index 65a08ed..58509c5 100644 --- a/sys/netipsec/xform.h +++ b/sys/netipsec/xform.h @@ -105,7 +105,7 @@ struct cryptoini; /* XF_IP4 */ extern int ip4_input6(struct mbuf **m, int *offp, int proto); -extern void ip4_input(struct mbuf *m, ...); +extern void ip4_input(struct mbuf *m, int); extern int ipip_output(struct mbuf *, struct ipsecrequest *, struct mbuf **, int, int); |