From 36c11f6033908a66b28f54df49e64849ea3958f0 Mon Sep 17 00:00:00 2001 From: Zdenek Styblik Date: Sun, 8 Dec 2013 19:06:13 +0000 Subject: [PATCH] ID: 285 - Fix SEGV in ipmi_lanplus_open_session If ipmi_lanplus_send_payload fails we get a NULL pointer returned. Error out straight away instead of continuing on and getting a SEGV when we dereference rsp. Commit for Anton Blanchard --- ipmitool/src/plugins/lanplus/lanplus.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/ipmitool/src/plugins/lanplus/lanplus.c b/ipmitool/src/plugins/lanplus/lanplus.c index 1f8169e..2b66361 100644 --- a/ipmitool/src/plugins/lanplus/lanplus.c +++ b/ipmitool/src/plugins/lanplus/lanplus.c @@ -2858,7 +2858,10 @@ ipmi_lanplus_open_session(struct ipmi_intf * intf) free(msg); msg = NULL; - + if (!rsp) { + lprintf(LOG_WARNING, "Error sending open session message."); + return -1; + } if (verbose) lanplus_dump_open_session_response(rsp);