summaryrefslogtreecommitdiff
path: root/src/soc/intel/alderlake/retimer.c
diff options
context:
space:
mode:
authorEran Mitrani <mitrani@google.com>2022-06-17 16:19:38 -0700
committerFelix Held <felix-coreboot@felixheld.de>2022-06-23 17:43:00 +0000
commit68033c2483e8c639641775acd486bc8fccc2a395 (patch)
treeafc157ab16b8b79abfefe539bbe05bbaa3f105a6 /src/soc/intel/alderlake/retimer.c
parentf23cf44c4715575ffdac41ad3ae0a5f65626ef09 (diff)
soc/intel/adl: Cast size in systemagent.c to fix overflow
This CL fixes my previous CL (commit ca741055e) which introduced a couple of issues found by Coverity (see below). The Coverity explanation is: "Potentially overflowing expression "size_field * 1048576U" with type "unsigned int" (32 bits, unsigned) is evaluated using 32-bit arithmetic, and then used in a context that expects an expression of type "uint64_t" (64 bits, unsigned)." *** CID 1490122: Integer handling issues (OVERFLOW_BEFORE_WIDEN) /src/soc/intel/alderlake/systemagent.c: 305 in get_dpr_size() *** CID 1490121: Integer handling issues (OVERFLOW_BEFORE_WIDEN) /src/soc/intel/alderlake/systemagent.c: 254 in get_dsm_size() BUG=b:149830546 BRANCH=firmware-brya-14505.B TEST='emerge-brya coreboot chromeos-bootimage' builds correctly. Tested on an Anahera device which successfully boots to ChromeOS with kernel version 5.10.109-15688-g857e654d1705. Change-Id: Ib2d66ad24a5ad67b51036ad376a6938f698134c3 Signed-off-by: Eran Mitrani <mitrani@google.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/65212 Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/soc/intel/alderlake/retimer.c')
0 files changed, 0 insertions, 0 deletions