diff options
| -rw-r--r-- | meta/recipes-core/ovmf/ovmf/0001-MdePkg-Fix-overflow-issue-in-BasePeCoffLib.patch | 36 | ||||
| -rw-r--r-- | meta/recipes-core/ovmf/ovmf_git.bb | 1 |
2 files changed, 37 insertions, 0 deletions
diff --git a/meta/recipes-core/ovmf/ovmf/0001-MdePkg-Fix-overflow-issue-in-BasePeCoffLib.patch b/meta/recipes-core/ovmf/ovmf/0001-MdePkg-Fix-overflow-issue-in-BasePeCoffLib.patch new file mode 100644 index 0000000000..c6e15c5069 --- /dev/null +++ b/meta/recipes-core/ovmf/ovmf/0001-MdePkg-Fix-overflow-issue-in-BasePeCoffLib.patch | |||
| @@ -0,0 +1,36 @@ | |||
| 1 | From 5f7bd3f3c4747d5bb2733f017f8c5b93b63a74e3 Mon Sep 17 00:00:00 2001 | ||
| 2 | From: Doug Flick <dougflick@microsoft.com> | ||
| 3 | Date: Fri, 22 Nov 2024 13:03:33 +0800 | ||
| 4 | Subject: [PATCH] MdePkg: Fix overflow issue in BasePeCoffLib | ||
| 5 | |||
| 6 | The RelocDir->Size is a UINT32 value, and RelocDir->VirtualAddress is | ||
| 7 | also a UINT32 value. The current code does not check for overflow when | ||
| 8 | adding RelocDir->Size to RelocDir->VirtualAddress. This patch adds a | ||
| 9 | check to ensure that the addition does not overflow. | ||
| 10 | |||
| 11 | Signed-off-by: Doug Flick <dougflick@microsoft.com> | ||
| 12 | Authored-by: sriraamx gobichettipalayam <sri..@intel.com> | ||
| 13 | |||
| 14 | CVE: CVE-2024-38796 | ||
| 15 | Upstream-Status: Backport [https://github.com/tianocore/edk2/commit/c95233b8525ca6828921affd1496146cff262e65] | ||
| 16 | Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com> | ||
| 17 | --- | ||
| 18 | MdePkg/Library/BasePeCoffLib/BasePeCoff.c | 2 +- | ||
| 19 | 1 file changed, 1 insertion(+), 1 deletion(-) | ||
| 20 | |||
| 21 | diff --git a/MdePkg/Library/BasePeCoffLib/BasePeCoff.c b/MdePkg/Library/BasePeCoffLib/BasePeCoff.c | ||
| 22 | index 86ff2e7..128090d 100644 | ||
| 23 | --- a/MdePkg/Library/BasePeCoffLib/BasePeCoff.c | ||
| 24 | +++ b/MdePkg/Library/BasePeCoffLib/BasePeCoff.c | ||
| 25 | @@ -1054,7 +1054,7 @@ PeCoffLoaderRelocateImage ( | ||
| 26 | RelocDir = &Hdr.Te->DataDirectory[0]; | ||
| 27 | } | ||
| 28 | |||
| 29 | - if ((RelocDir != NULL) && (RelocDir->Size > 0)) { | ||
| 30 | + if ((RelocDir != NULL) && (RelocDir->Size > 0) && (RelocDir->Size - 1 < MAX_UINT32 - RelocDir->VirtualAddress)) { | ||
| 31 | RelocBase = (EFI_IMAGE_BASE_RELOCATION *)PeCoffLoaderImageAddress (ImageContext, RelocDir->VirtualAddress, TeStrippedOffset); | ||
| 32 | RelocBaseEnd = (EFI_IMAGE_BASE_RELOCATION *)PeCoffLoaderImageAddress ( | ||
| 33 | ImageContext, | ||
| 34 | -- | ||
| 35 | 2.34.1 | ||
| 36 | |||
diff --git a/meta/recipes-core/ovmf/ovmf_git.bb b/meta/recipes-core/ovmf/ovmf_git.bb index 35ca8d1834..cc2ac4268c 100644 --- a/meta/recipes-core/ovmf/ovmf_git.bb +++ b/meta/recipes-core/ovmf/ovmf_git.bb | |||
| @@ -24,6 +24,7 @@ SRC_URI = "gitsm://github.com/tianocore/edk2.git;branch=master;protocol=https \ | |||
| 24 | file://0002-BaseTools-makefile-adjust-to-build-in-under-bitbake.patch \ | 24 | file://0002-BaseTools-makefile-adjust-to-build-in-under-bitbake.patch \ |
| 25 | file://0003-debug-prefix-map.patch \ | 25 | file://0003-debug-prefix-map.patch \ |
| 26 | file://0004-reproducible.patch \ | 26 | file://0004-reproducible.patch \ |
| 27 | file://0001-MdePkg-Fix-overflow-issue-in-BasePeCoffLib.patch \ | ||
| 27 | " | 28 | " |
| 28 | 29 | ||
| 29 | PV = "edk2-stable202402" | 30 | PV = "edk2-stable202402" |
