Jacob Garber has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/32249
Change subject: nb/intel/sandybridge: Set uninitialized run length ......................................................................
nb/intel/sandybridge: Set uninitialized run length
If the entire array is zero, then the length of the longest zero run is the length of the array itself.
Found-by: Coverity Scan, CID 1229715 Signed-off-by: Jacob Garber jgarber1@ualberta.ca Change-Id: Id23292087b14182448d70117915fb044e9c579f7 --- M src/northbridge/intel/sandybridge/raminit_common.c 1 file changed, 1 insertion(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/49/32249/1
diff --git a/src/northbridge/intel/sandybridge/raminit_common.c b/src/northbridge/intel/sandybridge/raminit_common.c index 55df03b..5ffb0d8 100644 --- a/src/northbridge/intel/sandybridge/raminit_common.c +++ b/src/northbridge/intel/sandybridge/raminit_common.c @@ -1142,6 +1142,7 @@ ret.middle = sz / 2; ret.start = 0; ret.end = sz; + ret.length = sz; ret.all = 1; return ret; }