From f3a235e9770035d429e66da511edef24a2e88b6b Mon Sep 17 00:00:00 2001 From: Patrick Georgi Date: Sun, 22 Feb 2015 16:51:23 +0100 Subject: [PATCH] intel/broadwell: free local heap object No real harm done (thanks to our free() implementation), but let's do it right. Change-Id: Ib98d28aabc043dff5c288728c33490a79f09f35a Signed-off-by: Patrick Georgi Found-by: Coverity Scan Reviewed-on: http://review.coreboot.org/8511 Reviewed-by: Paul Menzel Tested-by: build bot (Jenkins) Reviewed-by: Alexandru Gagniuc --- 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 fd28cfe5e6..9ec1d44bcd 100644 --- a/src/soc/intel/broadwell/me.c +++ b/src/soc/intel/broadwell/me.c @@ -998,6 +998,7 @@ static int intel_me_read_mbp(me_bios_payload *mbp_data, device_t dev) } #undef ASSIGN_FIELD_PTR + free(mbp); return 0; mbp_failure: