From patchwork Sun Mar 22 09:57:29 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Heinrich Schuchardt X-Patchwork-Id: 244083 List-Id: U-Boot discussion From: xypron.glpk at gmx.de (Heinrich Schuchardt) Date: Sun, 22 Mar 2020 10:57:29 +0100 Subject: [PATCH v2 0/2] efi_loader: correct reported length in GetNextVariable() Message-ID: <20200322095731.76982-1-xypron.glpk@gmx.de> The runtime service GetNextVariable() returns the length of the next variable including the closing 0x0000. This length should be in bytes. Comparing the output of EDK2 and U-Boot shows that this is currently not correctly implemented: EDK2: OsIndicationsSupported: 46 PlatformLang: 26 PlatformLangCodes: 36 U-Boot: OsIndicationsSupported: 23 PlatformLang: 13 PlatformLangCodes: 18 Provide correct length in GetNextVariable(). v2: correct return value of the initial call too increase buffer size for variable names in unit test Heinrich Schuchardt (2): efi_loader: correct reported length in GetNextVariable() efi_selftest: check length report by GetNextVariableName() lib/efi_loader/efi_variable.c | 11 ++++++----- lib/efi_selftest/efi_selftest_variables.c | 10 ++++++++-- 2 files changed, 14 insertions(+), 7 deletions(-) --- 2.25.1