deepblue/ipmitool
1
0
Fork 0
mirror of https://github.com/ipmitool/ipmitool.git synced 2025-05-10 18:47:22 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

session: Fix buffer overflow in ipmi_get_session_info

Browse Source 
Partial fix for CVE-2020-5208, see
https://github.com/ipmitool/ipmitool/security/advisories/GHSA-g659-9qxw-p7cp

The `ipmi_get_session_info` function does not properly check the
response `data_len`, which is used as a copy size, allowing stack buffer
overflow.
This commit is contained in:
Chrostoper Ertl 2019-11-28 16:51:49 +00:00 committed by Alexander Amelkin
parent 840fb1cbb4
commit 41d7026946
No known key found for this signature in database
GPG Key ID: E893587B5B74178D
1 changed files with 8 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
lib/ipmi_session.c
View File

@ -309,8 +309,10 @@ ipmi_get_session_info(struct ipmi_intf * intf,
} }
else else
{ {
memcpy(&session_info, rsp->data, rsp->data_len); memcpy(&session_info, rsp->data,
print_session_info(&session_info, rsp->data_len); __min(rsp->data_len, sizeof(session_info)));
print_session_info(&session_info,
__min(rsp->data_len, sizeof(session_info)));
} }
break; break;
@ -341,8 +343,10 @@ ipmi_get_session_info(struct ipmi_intf * intf,
break; break;
} }
memcpy(&session_info, rsp->data, rsp->data_len); memcpy(&session_info, rsp->data,
print_session_info(&session_info, rsp->data_len); __min(rsp->data_len, sizeof(session_info)));
print_session_info(&session_info,
__min(rsp->data_len, sizeof(session_info)));
} while (i <= session_info.session_slot_count); } while (i <= session_info.session_slot_count);
break; break;