grub2/SOURCES/0220-kern-file-Do-not-leak-device_name-on-error-in-grub_f.patch

40 lines
1.2 KiB
Diff
Raw Normal View History

2022-06-16 13:18:30 +00:00
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
From: Daniel Axtens <dja@axtens.net>
Date: Fri, 25 Jun 2021 02:19:05 +1000
Subject: [PATCH] kern/file: Do not leak device_name on error in
grub_file_open()
If we have an error in grub_file_open() before we free device_name, we
will leak it.
Free device_name in the error path and null out the pointer in the good
path once we free it there.
Signed-off-by: Daniel Axtens <dja@axtens.net>
Reviewed-by: Daniel Kiper <daniel.kiper@oracle.com>
(cherry picked from commit 1499a5068839fa37cb77ecef4b5bdacbd1ed12ea)
---
grub-core/kern/file.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/grub-core/kern/file.c b/grub-core/kern/file.c
2023-04-11 14:25:15 +00:00
index ec10e54fc0..db938e099d 100644
2022-06-16 13:18:30 +00:00
--- a/grub-core/kern/file.c
+++ b/grub-core/kern/file.c
2023-04-11 14:25:15 +00:00
@@ -84,6 +84,7 @@ grub_file_open (const char *name, enum grub_file_type type)
2022-06-16 13:18:30 +00:00
device = grub_device_open (device_name);
grub_free (device_name);
+ device_name = NULL;
if (! device)
goto fail;
2023-04-11 14:25:15 +00:00
@@ -138,6 +139,7 @@ grub_file_open (const char *name, enum grub_file_type type)
2022-06-16 13:18:30 +00:00
return file;
fail:
+ grub_free (device_name);
if (device)
grub_device_close (device);