diff options
| author | Peter Jones <pjones@redhat.com> | 2025-01-18 13:37:02 -0500 |
|---|---|---|
| committer | Peter Jones <pjones@redhat.com> | 2025-01-18 14:30:15 -0500 |
| commit | 3caa75e5693368f44478a1da272095cebc0b4c78 (patch) | |
| tree | 4c9c08931ae52cb085ca2870626f05af7aeb0882 /test-mok-mirror.c | |
| parent | 70366a286552760863bacb521fb00c654586b494 (diff) | |
| download | efi-boot-shim-3caa75e5693368f44478a1da272095cebc0b4c78.tar.gz efi-boot-shim-3caa75e5693368f44478a1da272095cebc0b4c78.zip | |
test-mok-mirror: minor bug fix
In 70366a286552760863bacb521fb00c654586b494, I introduced a test case
for test-mok-mirror to test the behavior when SetVariable() gives
EFI_OUT_OF_RESOURCES. Unfortunately this includes a memory error in its
cleanup functions:
==1972634== Invalid read of size 8
==1972634== at 0x4032F3: mock_sv_adjust_usage_data (mock-variables.c:468)
==1972634== by 0x40387B: mock_delete_variable (mock-variables.c:541)
==1972634== by 0x4014E8: mock_reset_variables (mock-variables.c:1353)
==1972634== by 0x401348: mock_finalize_vars_and_configs (mock-variables.c:1405)
==1972634== by 0x401731: main (test-mok-mirror.c:688)
==1972634== Address 0x1ffeffc4f8 is on thread 1's stack
==1972634== 6784 bytes below stack pointer
This is caused because the test data uses a linked-list data structure
in an array in the test data, and we try to iterate the list to free the
data, but of course the list pointers aren't initialized because it's an
array.
Whoops.
This patch makes it so we don't try to clean up that list, because we
don't need to.
Signed-off-by: Peter Jones <pjones@redhat.com>
Diffstat (limited to 'test-mok-mirror.c')
| -rw-r--r-- | test-mok-mirror.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/test-mok-mirror.c b/test-mok-mirror.c index 97b92959..0c45cead 100644 --- a/test-mok-mirror.c +++ b/test-mok-mirror.c @@ -633,6 +633,8 @@ test_mok_mirror_setvar_out_of_resources(void) test_mok_config_table, EFI_OUT_OF_RESOURCES); + mock_set_default_usage_limits(); + mock_set_variable_post_hook = NULL; mock_get_variable_post_hook = NULL; return ret; |