summaryrefslogtreecommitdiff
path: root/src/cryptsetup
diff options
context:
space:
mode:
authorZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>2023-01-24 22:45:25 +0100
committerLuca Boccassi <luca.boccassi@gmail.com>2023-01-24 23:07:21 +0000
commit2f6c52b919dcc4a52e597ada11af9267e3550029 (patch)
tree0fdfdbfa485ab2319704d96bf3d1fe6dce8cbd3b /src/cryptsetup
parent1d62f3a7a112c30399698dc76221f5e0e5b27ab5 (diff)
downloadsystemd-2f6c52b919dcc4a52e597ada11af9267e3550029.tar.gz
shared/efi-loader: fix compilation with !ENABLE_EFI, improve messages
When compiled without ENABLE_EFI, efi_stub_measured() was not defined, so compilation would fail. But it's not enough to add a stub that returns -EOPNOTSUPP. We call this function in various places and usually print the error at warning or error level, so we'd print a confusing message. We also can't add a stub that always returns 0, because then we'd print a message like "Kernel stub did not measure", which would be confusing too. Adding special handling for -EOPNOTSUPP in every caller is also unattractive. So instead efi_stub_measured() is reworked to log the warning or error internally, and such logging is removed from the callers, and a stub is added that logs a custom message.
Diffstat (limited to 'src/cryptsetup')
-rw-r--r--src/cryptsetup/cryptsetup.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/src/cryptsetup/cryptsetup.c b/src/cryptsetup/cryptsetup.c
index 38ee7f8935..819d5bbb97 100644
--- a/src/cryptsetup/cryptsetup.c
+++ b/src/cryptsetup/cryptsetup.c
@@ -828,9 +828,9 @@ static int measure_volume_key(
return 0;
}
- r = efi_stub_measured();
+ r = efi_stub_measured(LOG_WARNING);
if (r < 0)
- return log_warning_errno(r, "Failed to detect if we are running on a kernel image with TPM measurement enabled: %m");
+ return r;
if (r == 0) {
log_debug("Kernel stub did not measure kernel image into the expected PCR, skipping userspace measurement, too.");
return 0;