From 564aef2aff5a0ed9b0c4024c82f846c8633253f4 Mon Sep 17 00:00:00 2001 From: Hnan-Ting Lo Date: Wed, 30 May 2018 23:31:24 +0800 Subject: [PATCH] fru: internaluse: Fix segmentation fault (#9) A segmentation fault was trigged by executing 'ipmitool fru internaluse 1 info'. A memset() was attempted on the address of a pointer, not on the pointer itself. The erroneous memset() is removed with this commit because it duplicates the memset() in line 4072. Resolves ipmitool/ipmitool#9 Signed-off-by: Kylix Lo --- lib/ipmi_fru.c | 1 - 1 file changed, 1 deletion(-) diff --git a/lib/ipmi_fru.c b/lib/ipmi_fru.c index 42c1f19..6115ae6 100644 --- a/lib/ipmi_fru.c +++ b/lib/ipmi_fru.c @@ -4095,7 +4095,6 @@ ipmi_fru_get_internal_use_info( struct ipmi_intf * intf, return -1; } - memset(&fru, 0, sizeof(fru)); fru->size = (rsp->data[1] << 8) | rsp->data[0]; fru->access = rsp->data[2] & 0x1;