e016790fe1
- Security fixes and hardenings for boothole 3 / boothole 2022 (bsc#1198581) * 0001-video-Remove-trailing-whitespaces.patch * 0002-loader-efi-chainloader-Simplify-the-loader-state.patch * 0003-commands-boot-Add-API-to-pass-context-to-loader.patch - Fix CVE-2022-28736 (bsc#1198496) * 0004-loader-efi-chainloader-Use-grub_loader_set_ex.patch - Fix CVE-2022-28735 (bsc#1198495) * 0005-kern-efi-sb-Reject-non-kernel-files-in-the-shim_lock.patch * 0006-kern-file-Do-not-leak-device_name-on-error-in-grub_f.patch * 0007-video-readers-png-Abort-sooner-if-a-read-operation-f.patch * 0008-video-readers-png-Refuse-to-handle-multiple-image-he.patch - Fix CVE-2021-3695 (bsc#1191184) * 0009-video-readers-png-Drop-greyscale-support-to-fix-heap.patch - Fix CVE-2021-3696 (bsc#1191185) * 0010-video-readers-png-Avoid-heap-OOB-R-W-inserting-huff-.patch * 0011-video-readers-png-Sanity-check-some-huffman-codes.patch * 0012-video-readers-jpeg-Abort-sooner-if-a-read-operation-.patch * 0013-video-readers-jpeg-Do-not-reallocate-a-given-huff-ta.patch * 0014-video-readers-jpeg-Refuse-to-handle-multiple-start-o.patch - Fix CVE-2021-3697 (bsc#1191186) * 0015-video-readers-jpeg-Block-int-underflow-wild-pointer-.patch * 0016-normal-charset-Fix-array-out-of-bounds-formatting-un.patch - Fix CVE-2022-28733 (bsc#1198460) * 0017-net-ip-Do-IP-fragment-maths-safely.patch * 0018-net-netbuff-Block-overly-large-netbuff-allocs.patch * 0019-net-dns-Fix-double-free-addresses-on-corrupt-DNS-res.patch * 0020-net-dns-Don-t-read-past-the-end-of-the-string-we-re-.patch * 0021-net-tftp-Prevent-a-UAF-and-double-free-from-a-failed.patch * 0022-net-tftp-Avoid-a-trivial-UAF.patch * 0023-net-http-Do-not-tear-down-socket-if-it-s-already-bee.patch OBS-URL: https://build.opensuse.org/request/show/981228 OBS-URL: https://build.opensuse.org/package/show/Base:System/grub2?expand=0&rev=416
84 lines
2.7 KiB
Diff
84 lines
2.7 KiB
Diff
From e0cc6a601865a72cfe316f2cbbaaefcdd2ad8c69 Mon Sep 17 00:00:00 2001
|
|
From: Chris Coulson <chris.coulson@canonical.com>
|
|
Date: Tue, 5 Apr 2022 11:48:58 +0100
|
|
Subject: [PATCH 04/32] loader/efi/chainloader: Use grub_loader_set_ex()
|
|
|
|
This ports the EFI chainloader to use grub_loader_set_ex() in order to fix
|
|
a use-after-free bug that occurs when grub_cmd_chainloader() is executed
|
|
more than once before a boot attempt is performed.
|
|
|
|
Fixes: CVE-2022-28736
|
|
|
|
Signed-off-by: Chris Coulson <chris.coulson@canonical.com>
|
|
Reviewed-by: Daniel Kiper <daniel.kiper@oracle.com>
|
|
---
|
|
grub-core/loader/efi/chainloader.c | 15 +++++++--------
|
|
1 file changed, 7 insertions(+), 8 deletions(-)
|
|
|
|
diff --git a/grub-core/loader/efi/chainloader.c b/grub-core/loader/efi/chainloader.c
|
|
index 1ec09a166c..b3e1e89302 100644
|
|
--- a/grub-core/loader/efi/chainloader.c
|
|
+++ b/grub-core/loader/efi/chainloader.c
|
|
@@ -54,7 +54,6 @@ GRUB_MOD_LICENSE ("GPLv3+");
|
|
static grub_dl_t my_mod;
|
|
|
|
static grub_ssize_t fsize;
|
|
-static grub_efi_handle_t image_handle;
|
|
static grub_ssize_t cmdline_len;
|
|
static grub_efi_handle_t dev_handle;
|
|
|
|
@@ -64,8 +63,9 @@ static grub_efi_status_t (*entry_point) (grub_efi_handle_t image_handle, grub_e
|
|
#endif
|
|
|
|
static grub_err_t
|
|
-grub_chainloader_unload (void)
|
|
+grub_chainloader_unload (void *context)
|
|
{
|
|
+ grub_efi_handle_t image_handle = (grub_efi_handle_t) context;
|
|
grub_efi_loaded_image_t *loaded_image;
|
|
grub_efi_boot_services_t *b;
|
|
|
|
@@ -83,8 +83,9 @@ grub_chainloader_unload (void)
|
|
}
|
|
|
|
static grub_err_t
|
|
-grub_chainloader_boot (void)
|
|
+grub_chainloader_boot (void *context)
|
|
{
|
|
+ grub_efi_handle_t image_handle = (grub_efi_handle_t) context;
|
|
grub_efi_boot_services_t *b;
|
|
grub_efi_status_t status;
|
|
grub_efi_uintn_t exit_data_size;
|
|
@@ -644,6 +645,7 @@ grub_cmd_chainloader (grub_command_t cmd __attribute__ ((unused)),
|
|
grub_efi_physical_address_t address = 0;
|
|
grub_efi_uintn_t pages = 0;
|
|
grub_efi_char16_t *cmdline = NULL;
|
|
+ grub_efi_handle_t image_handle = NULL;
|
|
|
|
if (argc == 0)
|
|
return grub_error (GRUB_ERR_BAD_ARGUMENT, N_("filename expected"));
|
|
@@ -856,7 +858,7 @@ grub_cmd_chainloader (grub_command_t cmd __attribute__ ((unused)),
|
|
efi_call_2 (b->free_pages, address, pages);
|
|
grub_free (file_path);
|
|
|
|
- grub_loader_set (grub_chainloader_boot, grub_chainloader_unload, 0);
|
|
+ grub_loader_set_ex (grub_chainloader_boot, grub_chainloader_unload, image_handle, 0);
|
|
return 0;
|
|
|
|
fail:
|
|
@@ -874,10 +876,7 @@ grub_cmd_chainloader (grub_command_t cmd __attribute__ ((unused)),
|
|
efi_call_2 (b->free_pages, address, pages);
|
|
|
|
if (image_handle != NULL)
|
|
- {
|
|
- efi_call_1 (b->unload_image, image_handle);
|
|
- image_handle = NULL;
|
|
- }
|
|
+ efi_call_1 (b->unload_image, image_handle);
|
|
|
|
grub_dl_unref (my_mod);
|
|
|
|
--
|
|
2.34.1
|
|
|