From ffef3a6cb4824fdec8b031cdc9571829e0078b47 Mon Sep 17 00:00:00 2001
From: chris <chris@openbsd.org>
Date: Wed, 26 Apr 2000 22:57:27 +0000
Subject: [PATCH] Remove redundant superuser check in SIOCSIFMTU (already done
 in if.c) and modify comments

---
 sys/net/if_vlan.c | 9 ++++-----
 1 file 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 {
-- 
2.20.1