diff options
| author | Patrick Georgi <patrick@georgi-clan.de> | 2015-02-22 16:51:23 +0100 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2015-02-25 14:05:45 +0100 |
| commit | f3a235e9770035d429e66da511edef24a2e88b6b (patch) | |
| tree | 4e328ef5ec7a03a1105250212db25895be73678a | |
| parent | bdd185a355c15a6f18c11cf173cdeb4f599059a6 (diff) | |
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 <patrick@georgi-clan.de>
Found-by: Coverity Scan
Reviewed-on: http://review.coreboot.org/8511
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Tested-by: build bot (Jenkins)
Reviewed-by: Alexandru Gagniuc <mr.nuke.me@gmail.com>
| -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 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: |