Fix return value check in TPM event log parsing
The return value of fdt helper functions was changed to bool, but it
wasn't updated in the event log parsing code. Fix this.
Signed-off-by: Balint Dobszay <balint.dobszay@arm.com>
Change-Id: Ib26c5231533357cd7136965bc6ea5cd09073a587
diff --git a/components/config/loader/sp/sp_config_loader.c b/components/config/loader/sp/sp_config_loader.c
index 0b8e716..7762ee5 100644
--- a/components/config/loader/sp/sp_config_loader.c
+++ b/components/config/loader/sp/sp_config_loader.c
@@ -232,16 +232,19 @@
uint64_t tpm_event_log_addr = 0;
uint32_t tpm_event_log_size = 0;
- rc = dt_get_u64(fdt, node, "tpm_event_log_addr", &tpm_event_log_addr);
- if (rc)
+ if (!dt_get_u64(fdt, node, "tpm_event_log_addr", &tpm_event_log_addr)) {
+ DMSG("error: tpm_event_log_addr is missing");
return false;
+ }
- rc = dt_get_u32(fdt, node, "tpm_event_log_size", &tpm_event_log_size);
- if (rc)
+ if (!dt_get_u32(fdt, node, "tpm_event_log_size", &tpm_event_log_size)) {
+ DMSG("error: tpm_event_log_size is missing");
return false;
+ }
if (!config_store_add(CONFIG_CLASSIFIER_BLOB, "EVENT_LOG", 0,
(void *)tpm_event_log_addr, tpm_event_log_size)) {
+ DMSG("error: failed to add event log to config store");
return false;
}
}