summaryrefslogtreecommitdiff
path: root/payloads/libpayload/include/cbfs_glue.h
diff options
context:
space:
mode:
authorYidi Lin <yidilin@chromium.org>2023-11-02 14:17:02 +0800
committerJulius Werner <jwerner@chromium.org>2023-11-07 19:25:23 +0000
commiteabdd0252a2c3f8fb03b83781019243d47737e80 (patch)
tree2b3665eb5e1cebfa09aaa7151d02378d1797a9ad /payloads/libpayload/include/cbfs_glue.h
parente91785dfd8fd29a2f39a1f0ceb143015536706fb (diff)
libpayload/libc/time: Fix possible overflow in multiplication
The value from raw_read_cntfrq_el0() could be large enough to cause overflow when multiplied by USECS_PER_SEC. To prevent this, both USECS_PER_SEC and hz can be reduced by dividing them by their GCD. This patch also modifies the return type of `timer_hz()` from `uint64_t` to `uint32_t`, assuming that in practice the timestamp counter should never be that fast. BUG=b:307790895 TEST=boot to kernel and check the timestamps from `cbmem` Change-Id: Ia55532490651fcf47128b83a8554751f050bcc89 Signed-off-by: Yidi Lin <yidilin@chromium.org> Reviewed-on: https://review.coreboot.org/c/coreboot/+/78888 Reviewed-by: Julius Werner <jwerner@chromium.org> Reviewed-by: Yu-Ping Wu <yupingso@google.com> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'payloads/libpayload/include/cbfs_glue.h')
0 files changed, 0 insertions, 0 deletions