diff options
| author | Martin Pieuchot <mpi@cvs.openbsd.org> | 2013-08-15 09:18:38 +0000 |
|---|---|---|
| committer | Martin Pieuchot <mpi@cvs.openbsd.org> | 2013-08-15 09:18:38 +0000 |
| commit | 0508b67c71c3b7e397e7618ed4cff4707126ab32 (patch) | |
| tree | 0c3f82758d165ce910994fc5bdd82756fd81323e /sys/net/if_vlan_var.h | |
| parent | cd0b640726145ac28136a057e3560caf0288127f (diff) | |
Stop defining SIOC{S,G}ETVLAN as SIOC{S,G}IFGENERIC because they
are two differents ioctls manipulating structures of a different
size.
In particular this prevents vlan(4) to handle pppoe(4)-specific
ioctls and vice versa, that could lead to smashing the stack of
the caller.
Issue originally found by David Hill when compiling ifconfig
with clang.
Note that this is an ABI break but we're riding a biger dragon
right now.
ok mikeb@, kettenis@, sthen@
Diffstat (limited to 'sys/net/if_vlan_var.h')
| -rw-r--r-- | sys/net/if_vlan_var.h | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/sys/net/if_vlan_var.h b/sys/net/if_vlan_var.h index a1d2679671b..ae65ddca217 100644 --- a/sys/net/if_vlan_var.h +++ b/sys/net/if_vlan_var.h @@ -1,4 +1,4 @@ -/* $OpenBSD: if_vlan_var.h,v 1.20 2010/06/03 16:15:00 naddy Exp $ */ +/* $OpenBSD: if_vlan_var.h,v 1.21 2013/08/15 09:18:37 mpi Exp $ */ /* * Copyright 1998 Massachusetts Institute of Technology @@ -95,8 +95,6 @@ struct vlanreq { char vlr_parent[IFNAMSIZ]; u_short vlr_tag; }; -#define SIOCSETVLAN SIOCSIFGENERIC -#define SIOCGETVLAN SIOCGIFGENERIC #ifdef _KERNEL extern int vlan_input(struct ether_header *eh, struct mbuf *m); |