summaryrefslogtreecommitdiff
path: root/sys/net
diff options
context:
space:
mode:
authorChris Cappuccio <chris@cvs.openbsd.org>2000-04-26 22:57:28 +0000
committerChris Cappuccio <chris@cvs.openbsd.org>2000-04-26 22:57:28 +0000
commit10e8c9c895045fb519ae81e7b3c8262607558503 (patch)
tree5888420bcf220b5474ea380916fdcf5f54d15175 /sys/net
parentfbbc0a6a98a5dd6db7a3a312343f08ef5de1d9c0 (diff)
Remove redundant superuser check in SIOCSIFMTU
(already done in if.c) and modify comments
Diffstat (limited to 'sys/net')
-rw-r--r--sys/net/if_vlan.c9
1 files changed, 4 insertions, 5 deletions
diff --git a/sys/net/if_vlan.c b/sys/net/if_vlan.c
index e9b76469364..2f41736c946 100644
--- a/sys/net/if_vlan.c
+++ b/sys/net/if_vlan.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: if_vlan.c,v 1.1 2000/04/26 19:03:11 chris Exp $ */
+/* $OpenBSD: if_vlan.c,v 1.2 2000/04/26 22:57:27 chris Exp $ */
/*
* Copyright 1998 Massachusetts Institute of Technology
*
@@ -498,12 +498,11 @@ vlan_ioctl(struct ifnet *ifp, u_long cmd, caddr_t data)
case SIOCSIFMTU:
/*
- * Set the interface MTU.
+ * XXX Set the interface MTU.
* This is bogus. The underlying interface might support
- * jumbo frames.
+ * jumbo frames. It would be nice to replace ETHERMTU
+ * with the parent interface's MTU in the following statement.
*/
- if ((error = suser(p->p_ucred, &p->p_acflag)) != 0)
- break;
if (ifr->ifr_mtu > ETHERMTU) {
error = EINVAL;
} else {