From 13145e5d18229b5c05b93d23b5243d7e4505e1f2 Mon Sep 17 00:00:00 2001
From: Jakub Czapiga <jacz@semihalf.com>
Date: Tue, 6 Apr 2021 18:27:30 +0200
Subject: [PATCH] tests/lib/malloc-test: Fix possible memory overrun

Coverity reported false-positive possible memory overrun
in setup_calloc_test(). Change memset address to use actual
buffer instead of pointer stored in symbol value in order
to silence Coverity.

Signed-off-by: Jakub Czapiga <jacz@semihalf.com>
Change-Id: I19f0718c657d565e515157e66367573e08f51254
Found-by: Coverity CID 1452005
Reviewed-on: https://review.coreboot.org/c/coreboot/+/52136
Reviewed-by: Julius Werner <jwerner@chromium.org>
Reviewed-by: Paul Fagerburg <pfagerburg@chromium.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
---
 tests/lib/malloc-test.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/lib/malloc-test.c b/tests/lib/malloc-test.c
index fefacbb92b..b70de06ae1 100644
--- a/tests/lib/malloc-test.c
+++ b/tests/lib/malloc-test.c
@@ -50,7 +50,7 @@ static int setup_test(void **state)
 
 static int setup_calloc_test(void **state)
 {
-	memset(&_heap, 0xFF, TEST_HEAP_SZ);
+	memset(_test_heap, 0xFF, TEST_HEAP_SZ);
 	return setup_test(state);
 }