Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45181 )
Change subject: security/intel/stm: Fix size_t printf format error
......................................................................
Patch Set 3:
I though, Elyes would comment, but I think this fix is incorrect, and also has nothing to do with the platform but the build environment.
There is a dedicated length modifier for `size_t`. From `man 3 printf`:
z A following integer conversion corresponds to a size_t or
ssize_t argument, or a following n conversion corresponds to a
pointer to a size_t argument.
If I am right, please revert, or fix it correctly.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/45181
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Id51730c883d8fb9e87183121deb49f5fdda0114e
Gerrit-Change-Number: 45181
Gerrit-PatchSet: 3
Gerrit-Owner: Eugene Myers
cedarhouse1@comcast.net
Gerrit-Reviewer: Benjamin Doron
benjamin.doron00@gmail.com
Gerrit-Reviewer: Eugene Myers
cedarhouse@comcast.net
Gerrit-Reviewer: Patrick Georgi
pgeorgi@google.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-Reviewer: ron minnich
rminnich@gmail.com
Gerrit-CC: Felix Held
felix-coreboot@felixheld.de
Gerrit-CC: HAOUAS Elyes
ehaouas@noos.fr
Gerrit-CC: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-Comment-Date: Wed, 30 Sep 2020 13:41:33 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment