aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authornichole <Nichole_Wang@wistron.com>2020-06-16 09:23:24 +0800
committerVasant Hegde <hegdevasant@linux.vnet.ibm.com>2020-06-23 16:04:07 +0530
commit74a7a87a43ad55aff019ca8d98bea406fee1e125 (patch)
tree547b411ce6098cedcd179e7b299d9d019f48057c
parent914ac2864cb7566daa26b322b897ef0af2b50ac0 (diff)
downloadskiboot-74a7a87a43ad55aff019ca8d98bea406fee1e125.zip
skiboot-74a7a87a43ad55aff019ca8d98bea406fee1e125.tar.gz
skiboot-74a7a87a43ad55aff019ca8d98bea406fee1e125.tar.bz2
platform/mihawk: Fix IPMI double-free
[ Upstream commit 68dc040a6540c218d20517764ff5d740a3626c55 ] The commit 6826095 ("platform/mihawk: support dynamic PCIe slot table") added the IPMI OEM command to communicate with BMC. We do the ipmi_free_msg(msg) twice that caused the Fast-reboot fail. This patch fixes it by removing the IPMI double-free bug to restore Fast-reboot. Signed-off-by: Nichole Wang <Nichole_Wang@wistron.com> Cc: skiboot-stable@lists.ozlabs.org # skiboot-6.6.x Cc: skiboot-stable@lists.ozlabs.org # skiboot-op940.x Fixes: commit 6826095 ("platform/mihawk: support dynamic PCIe slot table") Reviewed-by: Vasant Hegde <hegdevasant@linux.vnet.ibm.com> Signed-off-by: Oliver O'Halloran <oohall@gmail.com> Signed-off-by: Vasant Hegde <hegdevasant@linux.vnet.ibm.com>
-rw-r--r--platforms/astbmc/mihawk.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/platforms/astbmc/mihawk.c b/platforms/astbmc/mihawk.c
index f3669ff..54c2884 100644
--- a/platforms/astbmc/mihawk.c
+++ b/platforms/astbmc/mihawk.c
@@ -533,8 +533,6 @@ static void mihawk_init(void)
if (timeout_ms == 0)
break;
}
-
- ipmi_free_msg(ipmi_msg);
}
prlog(PR_DEBUG, "Mihawk: IPMI_RISERF_QUERY finish. riser_state: %02x"