diff options
| author | Julius Werner <jwerner@chromium.org> | 2020-12-21 17:17:02 -0800 |
|---|---|---|
| committer | Julius Werner <jwerner@chromium.org> | 2021-02-18 02:32:56 +0000 |
| commit | 10ee7fc03ac38dfabb3190b00db2db66d35f541a (patch) | |
| tree | 657dad180a1eaa5eb982d23b51cef5551a66bb36 /src/commonlib | |
| parent | 5779ca718c84207f8ba14daaf74aa919d0c36d0a (diff) | |
| download | coreboot-10ee7fc03ac38dfabb3190b00db2db66d35f541a.tar.xz | |
cbfs: Fix attribute tag printing in cbfs_find_attr()
Attribute tags are defined as hexadecimal constants, not decimal, so it
makes more sense to print them like that in error messages as well.
Signed-off-by: Julius Werner <jwerner@chromium.org>
Change-Id: I3a5a6a8c9b8d24e57633595fc47221a483d8593a
Reviewed-on: https://review.coreboot.org/c/coreboot/+/48836
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Diffstat (limited to 'src/commonlib')
| -rw-r--r-- | src/commonlib/bsd/cbfs_private.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/commonlib/bsd/cbfs_private.c b/src/commonlib/bsd/cbfs_private.c index 527860db75..91f81e0db4 100644 --- a/src/commonlib/bsd/cbfs_private.c +++ b/src/commonlib/bsd/cbfs_private.c @@ -173,13 +173,13 @@ const void *cbfs_find_attr(const union cbfs_mdata *mdata, uint32_t attr_tag, siz const uint32_t len = be32toh(attr->len); if (offset + len > end) { - ERROR("Attribute %s[%u] overflows end of metadata\n", + ERROR("Attribute %s[%x] overflows end of metadata\n", mdata->h.filename, tag); return NULL; } if (tag == attr_tag) { if (size_check && len != size_check) { - ERROR("Attribute %s[%u] size mismatch: %u != %zu\n", + ERROR("Attribute %s[%x] size mismatch: %u != %zu\n", mdata->h.filename, tag, len, size_check); return NULL; } |