diff options
author | Angel Pons <th3fanbus@gmail.com> | 2021-01-09 16:52:19 +0100 |
---|---|---|
committer | Angel Pons <th3fanbus@gmail.com> | 2021-01-10 23:03:17 +0000 |
commit | 08ba81b6e48c6a498efb1aa7d2f1f0f005384100 (patch) | |
tree | c0f672a74fd6f7933cf2b71189c1ebd4d40f631b /util/intelmetool/msr.c | |
parent | af03936679c93c65645179c394d59f1a9ca2515d (diff) |
nb/intel/gm45: Guard `CxDRBy_BOUND_SHIFT` macro parameters
Wrap `r` in parentheses to avoid unexpected behavior with compound
expressions. This prevents `CxDRBy_BOUND_MB(r+1, base)` from triggering
undefined behavior when `r = 2`, as the shift would be greater than 32.
Change-Id: I14235b2708ab502d842da677451c14203a469b45
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/49261
Reviewed-by: Nico Huber <nico.h@gmx.de>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'util/intelmetool/msr.c')
0 files changed, 0 insertions, 0 deletions