summaryrefslogtreecommitdiffstats
path: root/MdeModulePkg/Library/BootMaintenanceManagerUiLib
diff options
context:
space:
mode:
authorJames Bottomley <jejb@linux.ibm.com>2020-11-26 04:13:48 +0800
committermergify[bot] <37929162+mergify[bot]@users.noreply.github.com>2020-11-27 01:19:34 +0000
commit872f953262d68a11da7bc2fb3ded16df234b8700 (patch)
treee7fb3c875b822cb51927d82c293f4144b4fcbc98 /MdeModulePkg/Library/BootMaintenanceManagerUiLib
parent21f984cedec1c613218480bc3eb5e92349a7a812 (diff)
downloadedk2-872f953262d68a11da7bc2fb3ded16df234b8700.tar.gz
edk2-872f953262d68a11da7bc2fb3ded16df234b8700.tar.bz2
edk2-872f953262d68a11da7bc2fb3ded16df234b8700.zip
MdeModulePkg/VariablePolicyLib: Fix runtime panic in ValidateSetVariable()edk2-stable202011
The current variable policy is allocated by AllocatePool(), which is boot time only. This means that if you do any variable setting in the runtime, the policy has been freed. Ordinarily this isn't detected because freed memory is still there, but when you boot the Linux kernel, it's been remapped so the actual memory no longer exists in the memory map causing a page fault. Fix this by making it AllocateRuntimePool(). For SMM drivers, the platform DSC is responsible for resolving the MemoryAllocationLib class to the SmmMemoryAllocationLib instance. In the SmmMemoryAllocationLib instance, AllocatePool() and AllocateRuntimePool() are implemented identically. Therefore this change is a no-op when the RegisterVariablePolicy() function is built into an SMM driver. The fix affects runtime DXE drivers only. Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3092 Signed-off-by: James Bottomley <jejb@linux.ibm.com> Reviewed-by: Laszlo Ersek <lersek@redhat.com> Tested-by: Laszlo Ersek <lersek@redhat.com> Reviewed-by: Liming Gao <gaoliming@byosoft.com.cn>
Diffstat (limited to 'MdeModulePkg/Library/BootMaintenanceManagerUiLib')
0 files changed, 0 insertions, 0 deletions