Page MenuHomeFreeBSD

igb(4): Fix out-of-bounds register access on VFs
ClosedPublic

Authored by kgalazka on Oct 8 2025, 8:10 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Jul 5, 7:59 AM
Unknown Object (File)
Sat, Jul 4, 8:21 PM
Unknown Object (File)
Fri, Jul 3, 10:40 PM
Unknown Object (File)
Thu, Jun 25, 9:12 AM
Unknown Object (File)
Wed, Jun 24, 12:09 PM
Unknown Object (File)
Sun, Jun 21, 6:19 PM
Unknown Object (File)
Sat, Jun 20, 6:46 PM
Unknown Object (File)
Sat, Jun 20, 6:46 PM

Details

Summary

Virtual Functions have access to a limited number of registers,
and their bus space size is lower. Use KASSERT to detect out-of-bounds
access and eliminate them to avoid kernel panics in production
environment.

Signed-off-by: Krzysztof Galazka <krzysztof.galazka@intel.com>

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 67639
Build 64522: arc lint + arc unit

Event Timeline

Wake-on-Lan configuration is not available for VFs.

sys/dev/e1000/if_em.c
1585

I wish these checks against !sc->vf_ifp were replaced with something like a E1000_IS_VF() macro, but is that what that check is for?

4380

Interesting...?

green light from validation point of view

This revision is now accepted and ready to land.Oct 23 2025, 10:46 AM
sys/dev/e1000/if_em.c
1585

Yeah, I was thinking about some refactoring, but I would prefer to do it as separate patch if you don't mind.

erj added inline comments.
sys/dev/e1000/if_em.c
1585

Sure.