dany/u-boot-megous
1
0
Fork 0
mirror of https://xff.cz/git/u-boot/ synced 2025-10-31 02:15:45 +01:00
Code Issues Packages Projects Releases Wiki Activity

efi_loader: variable: attributes may not be changed if a variable exists

Browse Source 
If a variable already exists, efi_set_variable() should not change
the variable's attributes. This patch enforces it.

Signed-off-by: AKASHI Takahiro <takahiro.akashi@linaro.org>
Reviewed-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
This commit is contained in:
AKASHI Takahiro
2019-05-24 15:59:03 +09:00
committed by Heinrich Schuchardt
parent dbebae5ec7
commit a2c6983740
1 changed files with 11 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
lib/efi_loader/efi_variable.c
View File

@@ -451,12 +451,21 @@ efi_status_t EFIAPI efi_set_variable(u16 *variable_name,
if (val) { if (val) {
parse_attr(val, &attr); parse_attr(val, &attr);
if (attr & READ_ONLY) {
/* We should not free val */ /* We should not free val */
val = NULL; val = NULL;
if (attr & READ_ONLY) {
ret = EFI_WRITE_PROTECTED; ret = EFI_WRITE_PROTECTED;
goto out; goto out;
} }
/*
* attributes won't be changed
* TODO: take care of APPEND_WRITE once supported
*/
if (attr != attributes) {
ret = EFI_INVALID_PARAMETER;
goto out;
}
} }
val = malloc(2 * data_size + strlen("{ro,run,boot}(blob)") + 1); val = malloc(2 * data_size + strlen("{ro,run,boot}(blob)") + 1);