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

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

Checks

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

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

Patch
diff mbox series

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;
 	}