diff options
| author | Peter Jones <pjones@redhat.com> | 2025-02-18 12:55:05 -0500 |
|---|---|---|
| committer | Peter Jones <pjones@redhat.com> | 2025-02-24 15:26:20 -0500 |
| commit | e136e645d54983d2b5a89610d1565c23538ca1fa (patch) | |
| tree | bac48bcb63158d82fa62781d5ba653ad97661ce6 /mock-variables.c | |
| parent | 848667d0f3a99401d93c93b3af16b55e3fb28cea (diff) | |
| download | efi-boot-shim-e136e645d54983d2b5a89610d1565c23538ca1fa.tar.gz efi-boot-shim-e136e645d54983d2b5a89610d1565c23538ca1fa.zip | |
mock-variables: fix debugging printf format specifier oopsie
This debug printf in our mock variable test code, which isn't normally
enabled, has a missing comma at the end of the format specifier. This
causes __FILE__ to be part of the format specifier, which then means
we've got a missing parameter and also the types don't match up like
you'd hope.
This causes the most confusing segfaults.
Signed-off-by: Peter Jones <pjones@redhat.com>
Diffstat (limited to 'mock-variables.c')
| -rw-r--r-- | mock-variables.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/mock-variables.c b/mock-variables.c index 656670d5..81828560 100644 --- a/mock-variables.c +++ b/mock-variables.c @@ -391,7 +391,7 @@ mock_get_next_variable_name(UINTN *size, CHAR16 *name, EFI_GUID *guid) } #if (defined(SHIM_DEBUG) && SHIM_DEBUG != 0) if (result) { - printf("%s:%d:%s(): found:%d result:%p &result->guid:%p &result->list:%p\n" + printf("%s:%d:%s(): found:%d result:%p &result->guid:%p &result->list:%p\n", __FILE__, __LINE__-1, __func__, found, result, &result->guid, &result->list); printf("%s:%d:%s(): "GUID_FMT"-%s\n", |