Page MenuHomeFreeBSD

if_vxlan(4): Check the size of data available in mbuf before using them
ClosedPublic

Authored by zlei on Sep 29 2022, 2:08 PM.
Referenced Files
Unknown Object (File)
Sun, Oct 18, 4:16 PM
Unknown Object (File)
Tue, Oct 6, 12:29 AM
Unknown Object (File)
Tue, Oct 6, 12:29 AM
Unknown Object (File)
Sat, Oct 3, 8:34 PM
Unknown Object (File)
Mon, Sep 28, 2:10 AM
Unknown Object (File)
Sep 11 2026, 5:17 AM
Unknown Object (File)
Sep 10 2026, 9:09 PM
Unknown Object (File)
Sep 6 2026, 4:03 PM

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

zlei requested review of this revision.Sep 29 2022, 2:08 PM
sys/net/if_vxlan.c
2852

Check it lazily may be better, i.e. after checking if running status.

This looks sane to me. We really do have to make sure there's enough contiguous data before we access it.

sys/net/if_vxlan.c
2852

I'm not sure it's worth having this check. m_pullup() simply won't do anything if the mbuf already has sufficient contiguous space.

sys/net/if_vxlan.c
2852

VXLAN over wireguard or some other setups that have deeper tunnels should be rare.
The m_pullup() is not defined as a macro or an inline function, thus a pre-check should be valuable.

This revision is now accepted and ready to land.Sep 30 2022, 9:54 AM