diff options
| author | Patrick Georgi <pgeorgi@chromium.org> | 2016-10-18 19:46:33 +0200 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2016-10-19 21:26:49 +0200 |
| commit | b753eeb7afe0f51a07725b13a2f53ed10d7930bc (patch) | |
| tree | 4b67206716fab0d3c4125825d3e37e5d780ae7fc /src | |
| parent | 9c5fc62f96f3f98f5434216c5531722ef97ca03f (diff) | |
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.
Change-Id: I1d93a6f45b64445662daa95b51128140ad0a87e2
Signed-off-by: Patrick Georgi <pgeorgi@chromium.org>
Found-by: Coverity Scan #1260716
Reviewed-on: https://review.coreboot.org/17055
Tested-by: build bot (Jenkins)
Reviewed-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
Diffstat (limited to 'src')
| -rw-r--r-- | src/soc/intel/broadwell/me.c | 1 |
1 files changed, 1 insertions, 0 deletions
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; } |