efi_loader: Add size checks to efi_create_indexed_name()
Although the function description states the caller must provide a sufficient buffer, it's better to have in function checks that the destination buffer can hold the intended value. So let's add an extra argument with the buffer size and check that before doing any copying. Signed-off-by: Ilias Apalodimas <ilias.apalodimas@linaro.org> Reviewed-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
This commit is contained in:

committed by
Heinrich Schuchardt

parent
47d2b3b9c9
commit
fe179d7fb5
@@ -603,7 +603,7 @@ static int unicode_test_efi_create_indexed_name(struct unit_test_state *uts)
|
||||
u16 *pos;
|
||||
|
||||
memset(buf, 0xeb, sizeof(buf));
|
||||
pos = efi_create_indexed_name(buf, "Capsule", 0x0af9);
|
||||
pos = efi_create_indexed_name(buf, sizeof(buf), "Capsule", 0x0af9);
|
||||
|
||||
ut_asserteq_mem(expected, buf, sizeof(expected));
|
||||
ut_asserteq(pos - buf, u16_strnlen(buf, SIZE_MAX));
|
||||
|
Reference in New Issue
Block a user