diff mbox series

[trivial] ipmi-sel: Free ipmi_msg in error path

Message ID 20200304085721.10425-1-hegdevasant@linux.vnet.ibm.com
State Accepted
Headers show
Series [trivial] ipmi-sel: Free ipmi_msg in error path | expand

Checks

Context Check Description
snowpatch_ozlabs/apply_patch success Successfully applied on branch master (82aed17a5468aff6b600ee1694a10a60f942c018)
snowpatch_ozlabs/snowpatch_job_snowpatch-skiboot success Test snowpatch/job/snowpatch-skiboot on branch master
snowpatch_ozlabs/snowpatch_job_snowpatch-skiboot-dco success Signed-off-by present

Commit Message

Vasant Hegde March 4, 2020, 8:57 a.m. UTC 
Signed-off-by: Vasant Hegde <hegdevasant@linux.vnet.ibm.com>
---
 hw/ipmi/ipmi-sel.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Oliver O'Halloran March 30, 2020, 6:27 a.m. UTC | #1 
On Wed, Mar 4, 2020 at 7:57 PM Vasant Hegde
<hegdevasant@linux.vnet.ibm.com> wrote:
>
> Signed-off-by: Vasant Hegde <hegdevasant@linux.vnet.ibm.com>

Thanks, merged as 841874b666e9c3f9aa7de477e19d9d8fd0e08ee7
diff mbox series

Patch

diff --git a/hw/ipmi/ipmi-sel.c b/hw/ipmi/ipmi-sel.c
index 537bb3b38..2ed4179a6 100644
--- a/hw/ipmi/ipmi-sel.c
+++ b/hw/ipmi/ipmi-sel.c
@@ -324,6 +324,7 @@  static void ipmi_elog_poll(struct ipmi_msg *msg)
 
 	if (bmc_platform->sw->ipmi_oem_partial_add_esel == 0) {
 		prlog(PR_WARNING, "Dropped eSEL: BMC code is buggy/missing\n");
+		ipmi_sel_free_msg(msg);
 		return;
 	}