diff options
| author | Jonathan Zhang <jonzhang@fb.com> | 2020-10-23 15:20:22 -0700 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2020-10-27 06:13:32 +0000 |
| commit | 8f594b7319c395bc09c711de05ff53f181c0c43d (patch) | |
| tree | 7fcec103851afb02ad3a956c3e4675448605a0d9 /src/lib/cbmem_common.c | |
| parent | bd21476f99785b83f45ad4ec02ed4e140505f912 (diff) | |
cpu/x86/mtrr: fix OVERFLOW_BEFORE_WIDEN
Integer handling issues:
Potentially overflowing expression "1 << size_msb" with type "int"
(32 bits, signed) is evaluated using 32-bit arithmetic, and then
used in a context that expects an expression of type "uint64_t"
(64 bits, unsigned).
Fixes: CID 1435825 and 1435826
Signed-off-by: Jonathan Zhang <jonzhang@fb.com>
Change-Id: If859521b44d9ec3ea744c751501b75d24e3b69e8
Reviewed-on: https://review.coreboot.org/c/coreboot/+/46711
Reviewed-by: Marc Jones <marc@marcjonesconsulting.com>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/lib/cbmem_common.c')
0 files changed, 0 insertions, 0 deletions