[coreboot-gerrit] Change in ...coreboot[master]: nb/intel/sandybridge: Set uninitialized run length

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change and it was merged. ( 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 
Change-Id: Id23292087b14182448d70117915fb044e9c579f7
Reviewed-on: https://review.coreboot.org/c/coreboot/+/32249
Tested-by: build bot (Jenkins) 
Reviewed-by: Patrick Rudolph 
Reviewed-by: Paul Menzel 
---
M src/northbridge/intel/sandybridge/raminit_common.c
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Paul Menzel: Looks good to me, but someone else must approve
  Patrick Rudolph: Looks good to me, approved

diff --git a/src/northbridge/intel/sandybridge/raminit_common.c b/src/northbridge/intel/sandybridge/raminit_common.c
index 55df03b..cbbd231 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;
    }