Index | Thread | Search

From:
Ido <ido@wireplug.org>
Subject:
ether_input(): remove redundant etype assignment
To:
tech@openbsd.org
Date:
Mon, 16 Mar 2026 13:04:07 -0400

Download raw body.

Thread
etype is already assigned the same value early in the function and remains unchanged.

---
Ido

diff --git sys/net/if_ethersubr.c sys/net/if_ethersubr.c
index d67e791e7fa..63c4a80d811 100644
--- sys/net/if_ethersubr.c
+++ sys/net/if_ethersubr.c
@@ -525,7 +525,6 @@ ether_input(struct ifnet *ifp, struct mbuf *m, struct netstack *ns)
 	 * At this point it is known that the packet is destined
 	 * for layer 3 protocol handling on the local port.
 	 */
-	etype = ntohs(eh->ether_type);
 
 	switch (etype) {
 	case ETHERTYPE_IP: