]> git.proxmox.com Git - mirror_edk2.git/commit - MdeModulePkg/Library/DeviceManagerUiLib/DeviceManager.c
MdeModulePkg/DeviceManagerUiLib: connect all before creating menu page
authorArd Biesheuvel <ard.biesheuvel@arm.com>
Wed, 8 Apr 2020 17:23:27 +0000 (19:23 +0200)
committermergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Thu, 30 Apr 2020 07:51:10 +0000 (07:51 +0000)
commit13406bdeb5c5bea7f5cee6b3c0dfaff253579710
tree4c79519bcd9fc48ba18cf40523f69549960a72ac
parentf07fb43b2d3f92a15d6992205b72ba5df0e74fe2
MdeModulePkg/DeviceManagerUiLib: connect all before creating menu page

The device manager UI library creates a UiApp submenu that contains a
list of network devices in the system. The logic that creates this menu
assumes that all handles have been connected to their drivers, but this
is not guaranteed in the general case: due to the way UiApp is constructed,
this logic runs before the UiApp entrypoint is invoked, which is where
ConnectAll() is normally called to ensure that all existing controllers
are connected to their drivers. Moving this call into DeviceManagerUiLib
is not an option, since it is incorporated into UiApp via NULL library
class resolution, and so it may not be included to begin with.

So work around this by doing an explicit additional ConnectAll() before
populating the pages.

Signed-off-by: Ard Biesheuvel <ard.biesheuvel@arm.com>
Link: https://edk2.groups.io/g/devel/topic/72879609#57075
Reviewed-by: Dandan Bi <dandan.bi@intel.com>
MdeModulePkg/Library/DeviceManagerUiLib/DeviceManager.c
MdeModulePkg/Library/DeviceManagerUiLib/DeviceManager.h
MdeModulePkg/Library/DeviceManagerUiLib/DeviceManagerUiLib.inf