summaryrefslogtreecommitdiff
path: root/src/northbridge
diff options
context:
space:
mode:
authorJacob Garber <jgarber1@ualberta.ca>2019-04-08 22:21:43 -0600
committerPatrick Georgi <pgeorgi@google.com>2019-04-09 17:24:40 +0000
commite0c181d48790be12b469b175bf4d9931a8c79fa1 (patch)
tree445984afb6d83bdd180ced155cad0a022055e2f7 /src/northbridge
parentcc86d8921b7d285fd0b5841acfeca4ee12be3b3d (diff)
downloadcoreboot-e0c181d48790be12b469b175bf4d9931a8c79fa1.tar.xz
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 Reviewed-on: https://review.coreboot.org/c/coreboot/+/32249 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Patrick Rudolph <siro@das-labor.org> Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Diffstat (limited to 'src/northbridge')
-rw-r--r--src/northbridge/intel/sandybridge/raminit_common.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/src/northbridge/intel/sandybridge/raminit_common.c b/src/northbridge/intel/sandybridge/raminit_common.c
index 55df03b8d2..cbbd2317ed 100644
--- a/src/northbridge/intel/sandybridge/raminit_common.c
+++ b/src/northbridge/intel/sandybridge/raminit_common.c
@@ -1142,6 +1142,7 @@ static struct run get_longest_zero_run(int *seq, int sz)
ret.middle = sz / 2;
ret.start = 0;
ret.end = sz;
+ ret.length = sz;
ret.all = 1;
return ret;
}