From 4b058899a24a1688178bf0eaf071962e00076857 Mon Sep 17 00:00:00 2001 From: Jim Mankovich Date: Tue, 5 Jun 2012 16:53:09 +0000 Subject: [PATCH] Fix stack overflow in delloem setled From: Jan Safranek The buffer should be bigger to hold the 10 character, which are set later in the function. --- ipmitool/lib/ipmi_delloem.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ipmitool/lib/ipmi_delloem.c b/ipmitool/lib/ipmi_delloem.c index e9e17a0..9ecf427 100644 --- a/ipmitool/lib/ipmi_delloem.c +++ b/ipmitool/lib/ipmi_delloem.c @@ -4861,7 +4861,7 @@ CheckSetLEDSupport(struct ipmi_intf * intf) { struct ipmi_rs * rsp = NULL; struct ipmi_rq req = {0}; - uint8_t data[4]; + uint8_t data[10]; SetLEDSupported = 0; req.msg.netfn = DELL_OEM_NETFN;