if_bridge: Fix NULL softc dereference in bridge_input()HEADmain

In bridge_input, sc is initialized to NULL and doesn't get
resolved until after the Ethernet header pullup.
So the pullup's failure path ends up dereferencing the NULL sc
when bumping up IFCOUNTER_IERRORS.

The m_freem call right under it is redundant as the failure path in
m_pullup already freed the chain.

Drop both lines, matching what we have in bridge_output.

ether_input_internal() discards frames shorter than ETHER_HDR_LEN
before the bridge hook, so it is unlikely that it will fire.
We still keep the guard as lagg(4) and ng_ether(4) may replace
the mbuf before the bridge hook.

Signed-off-by:	Aaron Espinoza <acesp25@freebsd.org>
Reviewed by:	pouria
Pull Request:	https://github.com/freebsd/freebsd-src/pull/2393

0 files changed, 0 insertions, 0 deletions