[coreboot-gerrit] Change in coreboot[master]: sb/amd/{cimx,}/sb{700,800,900}: Prevent uninitialized reads

[Jacob Garber (Code Review)]

Hello Kyösti Mälkki, Angel Pons, David Hendricks, build bot (Jenkins), Patrick Georgi, Martin Roth,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/33790

to look at the new patch set (#2).

Change subject: sb/amd/{cimx,}/sb{700,800,900}: Prevent uninitialized reads
......................................................................

sb/amd/{cimx,}/sb{700,800,900}: Prevent uninitialized reads

There are two hard things in computer science: cache invalidation,
naming things, and off-by-one errors. -- Anonymous

var_num records the number of initialized entries in the reg_var array.
However, this means the index of the last initialized element is one
less than the value of var_num, so we need to take that into account
when indexing into the array. This has already been fixed in several
other places (eg. sb/amd/pi/hudson/lpc.c), so let's also do so here.

Change-Id: Ibefabaca42866a3f2b22eff979c73badf86ac317
Signed-off-by: Jacob Garber 
Found-by: scan-build 8.0.0
---
M src/southbridge/amd/cimx/sb800/lpc.c
M src/southbridge/amd/cimx/sb900/lpc.c
M src/southbridge/amd/sb700/lpc.c
M src/southbridge/amd/sb800/lpc.c
4 files changed, 12 insertions(+), 12 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/90/33790/2