[PATCH 1/1] efi_loader: efi_capsule_delete_file() NULL dereference

2 Jun
2021
2 Jun
'21
7:40 p.m.
If opening a file fails, the file handle pointer is not valid and must not be dereferenced.
Signed-off-by: Heinrich Schuchardt xypron.glpk@gmx.de --- lib/efi_loader/efi_capsule.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/lib/efi_loader/efi_capsule.c b/lib/efi_loader/efi_capsule.c index 9ead0d2c78..60309d4a07 100644 --- a/lib/efi_loader/efi_capsule.c +++ b/lib/efi_loader/efi_capsule.c @@ -901,7 +901,8 @@ static efi_status_t efi_capsule_delete_file(const u16 *filename) /* ignore an error */ EFI_CALL((*dirh->close)(dirh));
- ret = EFI_CALL((*fh->delete)(fh)); + if (ret == EFI_SUCCESS) + ret = EFI_CALL((*fh->delete)(fh));
return ret; } -- 2.30.2
1438
Age (days ago)
1438
Last active (days ago)
0 comments
1 participants
participants (1)
-
Heinrich Schuchardt