From: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
Date: Sun, 10 Jul 2022 13:46:57 +0000 (+0200)
Subject: efi_loader: memory leak in efi_set_bootdev()
X-Git-Url: http://git.dujemihanovic.xyz/?a=commitdiff_plain;h=868353daa379849953ae30f400536ff61624de54;p=u-boot.git

efi_loader: memory leak in efi_set_bootdev()

efi_dp_str() allocates memory which should be released after use.

Use %pD printf code. Adjust message wording.

Fixes: d837cb1e3b6b ("efi: Add debugging to efi_set_bootdev()")
Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
---

diff --git a/cmd/bootefi.c b/cmd/bootefi.c
index 37ce659fa1..8ab0ff5a64 100644
--- a/cmd/bootefi.c
+++ b/cmd/bootefi.c
@@ -119,9 +119,9 @@ void efi_set_bootdev(const char *dev, const char *devnr, const char *path,
 			efi_free_pool(image_tmp);
 		}
 		bootefi_image_path = image;
-		log_debug("- recorded device %ls\n", efi_dp_str(device));
+		log_debug("- boot device %pD\n", device);
 		if (image)
-			log_debug("- and image %ls\n", efi_dp_str(image));
+			log_debug("- image %pD\n", image);
 	} else {
 		log_debug("- efi_dp_from_name() failed, err=%lx\n", ret);
 		efi_clear_bootdev();