From 7e9cd85e8dab191594163ff730fbee453839a6fa Mon Sep 17 00:00:00 2001 From: Francois Isabelle Date: Wed, 7 Nov 2007 16:15:32 +0000 Subject: [PATCH] Added null check in to avoid segfault --- ipmitool/lib/ipmi_sol.c | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/ipmitool/lib/ipmi_sol.c b/ipmitool/lib/ipmi_sol.c index 4c2be29..3f75ad1 100644 --- a/ipmitool/lib/ipmi_sol.c +++ b/ipmitool/lib/ipmi_sol.c @@ -487,9 +487,13 @@ ipmi_get_sol_info( } break; case 0x80: - lprintf(LOG_ERR, "Info: SOL parameter '%s' not supported - defaulting to %d", - val2str(data[1], sol_parameter_vals), intf->session->port); - params->payload_port = intf->session->port; + if( intf->session != NULL ){ + lprintf(LOG_ERR, "Info: SOL parameter '%s' not supported - defaulting to %d", val2str(data[1], sol_parameter_vals), intf->session->port); + params->payload_port = intf->session->port; + } else { + lprintf(LOG_ERR, "Info: SOL parameter '%s' not supported - can't determine which payload port to use on NULL session" ); + return -1; + } break; default: lprintf(LOG_ERR, "Error requesting SOL parameter '%s': %s",