From d16e59ce44d4ea674267cf41179144fc1a497737 Mon Sep 17 00:00:00 2001 From: Patrick Georgi Date: Tue, 18 Oct 2016 19:46:33 +0200 Subject: [PATCH] UPSTREAM: intel/broadwell: "free" memory after use While we stub out free(), tools like coverity scan have no idea, and it might change in the future. So free it. BUG=None BRANCH=None TEST=None Signed-off-by: Patrick Georgi Found-by: Coverity Scan #1260716 Reviewed-on: https://review.coreboot.org/17055 Tested-by: build bot (Jenkins) Reviewed-by: Jonathan Neuschfer Change-Id: I1d93a6f45b64445662daa95b51128140ad0a87e2 Reviewed-on: https://chromium-review.googlesource.com/402369 Commit-Ready: Furquan Shaikh Tested-by: Furquan Shaikh Reviewed-by: Aaron Durbin --- src/soc/intel/broadwell/me.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/soc/intel/broadwell/me.c b/src/soc/intel/broadwell/me.c index 893c19bb8e..1b086ec94d 100644 --- a/src/soc/intel/broadwell/me.c +++ b/src/soc/intel/broadwell/me.c @@ -966,6 +966,7 @@ static int intel_me_read_mbp(me_bios_payload *mbp_data, device_t dev) } #undef ASSIGN_FIELD_PTR + free(mbp); return ret; }