Reporting Issues
Bug 3327 - DxeCore: Image-based MAT splitting may emit unintended permissions
Summary: DxeCore: Image-based MAT splitting may emit unintended permissions
Status: UNCONFIRMED
Alias: None
Product: EDK2
Classification: Unclassified
Component: Code (show other bugs)
Version: Current
Hardware: All All
: Lowest normal
Assignee: Marvin Häuser
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-04-12 11:16 UTC by Marvin Häuser
Modified: 2021-04-13 21:54 UTC (History)
4 users (show)

See Also:
EDK II Code First industry standard specifications: ---
Branch URL:
Release(s) the issue is observed: EDK II Master
The OS the target platform is running: ---
Package: MdeModulePkg
Release(s) the issues must be fixed: EDK II Master
Tianocore documents:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Marvin Häuser 2021-04-12 11:16:40 UTC
This issue has been identified and fixed for SMM before: https://github.com/tianocore/edk2/commit/026e2ca2c315ea0d19be5f72b89009bdcf5f10c8

Despite the fact that the DXE and SMM cores have basically the same code for the MAT split logic, this fix was never ported to the DXE code, which still assumes any non-PE memory is data[1].

Ideally, the PE memory covers the entire range and this does not matter. However, if this is agreed upon, this must be properly and consistently ensured.

[1] https://github.com/tianocore/edk2/blob/2072c22a0d63c780b0cc6377f6d4ffb116ad6144/MdeModulePkg/Core/Dxe/Misc/MemoryAttributesTable.c#L772-L788
Comment 1 gaoliming 2021-04-13 21:54:39 UTC
Marvin, you can continue to provide the proposal.