HomeFreeBSD

MFC r312544

Description

MFC r312544

Fix reference to free memory in ixgbe/if_media.c

When ixgbe receives an interrupt indicating that a new optical module
may have been inserted, it discards all of its current media types
by calling ifmedia_removeall() and then creates a new set of media
types for the supported media on the new module.  However,
ifmedia_removeall() was maintaining a pointer to whatever the
current media type was before the call to ifmedia_removealL().
The result of this was that any attempt to read the current media
type of the interface (e.g. via ifconfig) would return potentially
garbage data from free memory (or if one were particularly unlucky
on an architecture that does not malloc() from a direct map, page
fault the kernel).

Fix this by NULL'ing out the current media field in if_media.c,
and have ixgbe update the current media type after recreating
them.

Submitted by: Matt Joras <matt.joras AT gmail DOT com>
Reviewed by:  sbruno, erj
MFC after:    1 week
Sponsored by: Dell EMC Isilon
Differential Revision:        https://reviews.freebsd.org/D9164

Details

Provenance
rstoneAuthored on
Parents
rS313387: MFC r312544
Branches
Unknown
Tags
Unknown