diff mbox series

[U-Boot,1/1] efi_loader: call EFI_EXIT in efi_copy_mem, efi_set_mem

Message ID 20170921164130.31516-1-xypron.glpk@gmx.de
State Accepted, archived
Delegated to: Alexander Graf
Headers show
Series [U-Boot,1/1] efi_loader: call EFI_EXIT in efi_copy_mem, efi_set_mem | expand

Commit Message

Heinrich Schuchardt Sept. 21, 2017, 4:41 p.m. UTC 
EFI_ENTRY and EFI_EXIT calls must match.

Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
---
 lib/efi_loader/efi_boottime.c | 2 ++
 1 file changed, 2 insertions(+)

Comments

Simon Glass Sept. 25, 2017, 2:14 a.m. UTC | #1 
On 21 September 2017 at 10:41, Heinrich Schuchardt <xypron.glpk@gmx.de> wrote:
> EFI_ENTRY and EFI_EXIT calls must match.
>
> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
> ---
>  lib/efi_loader/efi_boottime.c | 2 ++
>  1 file changed, 2 insertions(+)

Reviewed-by: Simon Glass <sjg@chromium.org>
diff mbox series

Patch

diff --git a/lib/efi_loader/efi_boottime.c b/lib/efi_loader/efi_boottime.c
index cbc4007f7b..2c2620a46d 100644
--- a/lib/efi_loader/efi_boottime.c
+++ b/lib/efi_loader/efi_boottime.c
@@ -1882,6 +1882,7 @@  static void EFIAPI efi_copy_mem(void *destination, void *source,
 {
 	EFI_ENTRY("%p, %p, %ld", destination, source, length);
 	memcpy(destination, source, length);
+	EFI_EXIT(EFI_SUCCESS);
 }
 
 /*
@@ -1899,6 +1900,7 @@  static void EFIAPI efi_set_mem(void *buffer, unsigned long size, uint8_t value)
 {
 	EFI_ENTRY("%p, %ld, 0x%x", buffer, size, value);
 	memset(buffer, value, size);
+	EFI_EXIT(EFI_SUCCESS);
 }
 
 /*