summaryrefslogtreecommitdiff
path: root/src/security
diff options
context:
space:
mode:
authorArthur Heymans <arthur@aheymans.xyz>2021-06-25 17:09:30 +0200
committerWerner Zeh <werner.zeh@siemens.com>2021-06-28 04:25:23 +0000
commitbf4a8d637234ddcd4256cbc10669e83c53347491 (patch)
treefd016bfd44debf4514c0d8f65f5b86751ffac37c /src/security
parent483880e2255d3876428bd038c62d4f19e40c5aba (diff)
security/intel/cbnt: Fix logging
The wrong format was used. It looks like struct bitfields are of type int according to gcc so %u needs to be used and not %lu. Change-Id: I54419d722aec0d43e6f54a4bb4eb4d899c909fec Signed-off-by: Arthur Heymans <arthur@aheymans.xyz> Reviewed-on: https://review.coreboot.org/c/coreboot/+/55847 Reviewed-by: Angel Pons <th3fanbus@gmail.com> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/security')
-rw-r--r--src/security/intel/cbnt/logging.c26
1 files changed, 13 insertions, 13 deletions
diff --git a/src/security/intel/cbnt/logging.c b/src/security/intel/cbnt/logging.c
index 8e02202550..55354b4ce8 100644
--- a/src/security/intel/cbnt/logging.c
+++ b/src/security/intel/cbnt/logging.c
@@ -121,25 +121,25 @@ void intel_cbnt_log_registers(void)
{
const union sacm_info acm_info = { .msr = rdmsr(MSR_BOOT_GUARD_SACM_INFO) };
LOG("SACM INFO MSR (0x13A) raw: 0x%016llx\n", acm_info.raw);
- LOG(" NEM status: %ul\n", acm_info.nem_enabled);
+ LOG(" NEM status: %u\n", acm_info.nem_enabled);
LOG(" TPM type: %s\n", tpm_type[acm_info.tpm_type]);
- LOG(" TPM succes: %ul\n", acm_info.tpm_success);
- LOG(" FACB: %ul\n", acm_info.facb);
- LOG(" measured boot: %ul\n", acm_info.measured_boot);
- LOG(" verified boot: %ul\n", acm_info.verified_boot);
- LOG(" revoked: %ul\n", acm_info.revoked);
- LOG(" BtG capable: %ul\n", acm_info.btg_cap);
- LOG(" TXT capable: %ul\n", acm_info.txt_cap);
+ LOG(" TPM succes: %u\n", acm_info.tpm_success);
+ LOG(" FACB: %u\n", acm_info.facb);
+ LOG(" measured boot: %u\n", acm_info.measured_boot);
+ LOG(" verified boot: %u\n", acm_info.verified_boot);
+ LOG(" revoked: %u\n", acm_info.revoked);
+ LOG(" BtG capable: %u\n", acm_info.btg_cap);
+ LOG(" TXT capable: %u\n", acm_info.txt_cap);
const union cbnt_bootstatus btsts = {
.raw = read64p(CBNT_BOOTSTATUS),
};
LOG("BOOTSTATUS (0xA0) raw: 0x%016llx\n", btsts.raw);
- LOG(" Bios trusted: %ul\n", btsts.bios_trusted);
- LOG(" TXT disabled by policy: %ul\n", btsts.txt_dis_pol);
- LOG(" Bootguard startup error: %ul\n", btsts.btg_startup_err);
- LOG(" TXT ucode or ACM error: %ul\n", btsts.txt_err);
- LOG(" TXT measurement type 7: %ul\n", btsts.type7);
+ LOG(" Bios trusted: %u\n", btsts.bios_trusted);
+ LOG(" TXT disabled by policy: %u\n", btsts.txt_dis_pol);
+ LOG(" Bootguard startup error: %u\n", btsts.btg_startup_err);
+ LOG(" TXT ucode or ACM error: %u\n", btsts.txt_err);
+ LOG(" TXT measurement type 7: %u\n", btsts.type7);
const union cbnt_errorcode err = {
.raw = read32p(CBNT_ERRORCODE),