summaryrefslogtreecommitdiff
path: root/src/southbridge/amd/sb700/smbus.c
diff options
context:
space:
mode:
authorMartin Roth <martinroth@google.com>2016-01-06 21:18:21 -0700
committerMartin Roth <martinroth@google.com>2016-01-08 17:21:59 +0100
commitfecc24af049bfec00efce49858e228dd9ab2d11c (patch)
tree5daf202da30d149143f7d38213a16638cbce7272 /src/southbridge/amd/sb700/smbus.c
parentb95a0745865a084e5426b9930eba0dadd3301c23 (diff)
vendorcode/amd/agesa/f15tn: Fix out of bounds read on on memory voltage
I think this has a fairly low likelyhood of happening, but if AGESA can't determine the voltage of the memory, it assignes a value of 255 to the variable that it later uses to read from an 3-value array. There is an assert, but that doesn't halt AGESA, so it would use some random value. If the voltage can't be determined, fall back to 1.5v as the default value. Fixes coverity warning 1294803 - Out-of-bounds read Change-Id: Ib9e568175edbdf55a7a4c35055da7169ea7f2ede Signed-off-by: Martin Roth <martinroth@google.com> Reviewed-on: https://review.coreboot.org/12855 Tested-by: build bot (Jenkins) Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
Diffstat (limited to 'src/southbridge/amd/sb700/smbus.c')
0 files changed, 0 insertions, 0 deletions