diff options
author | Chris Cappuccio <chris@cvs.openbsd.org> | 2000-04-26 22:57:28 +0000 |
---|---|---|
committer | Chris Cappuccio <chris@cvs.openbsd.org> | 2000-04-26 22:57:28 +0000 |
commit | 10e8c9c895045fb519ae81e7b3c8262607558503 (patch) | |
tree | 5888420bcf220b5474ea380916fdcf5f54d15175 /sys/net | |
parent | fbbc0a6a98a5dd6db7a3a312343f08ef5de1d9c0 (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.c | 9 |
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 { |