diff options
| author | Ahmad Fatoum <a.fatoum@pengutronix.de> | 2021-11-22 09:47:27 +0100 |
|---|---|---|
| committer | Sascha Hauer <s.hauer@pengutronix.de> | 2021-12-13 23:19:38 +0100 |
| commit | de8c3d980c9031e3276043c7f9ee22f9990c929a (patch) | |
| tree | 76e00f5d5deea5f2ef2f860a980c5b5a26262734 /common | |
| parent | 24f8814bf1abf2535b828f870293fc2060beb754 (diff) | |
| download | barebox-de8c3d980c9031e3276043c7f9ee22f9990c929a.tar.gz barebox-de8c3d980c9031e3276043c7f9ee22f9990c929a.tar.xz | |
efi: don't zero executable buffer before freeing
For security, it sometimes makes sense to zero memory before freeing.
This is not the case here, so remove the memset.
Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
Link: https://lore.barebox.org/20211122084732.2597109-26-a.fatoum@pengutronix.de
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Diffstat (limited to 'common')
| -rw-r--r-- | common/efi/payload/image.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/common/efi/payload/image.c b/common/efi/payload/image.c index 3bcb8fce2d..c1206cd6e2 100644 --- a/common/efi/payload/image.c +++ b/common/efi/payload/image.c @@ -106,7 +106,6 @@ static int efi_load_image(const char *file, efi_loaded_image_t **loaded_image, *h = handle; out: - memset(exe, 0, size); free(exe); return -efi_errno(efiret); } |