admin/ipmitool/patches/0008-CVE-2020-5208-session-Fix-buffer-overflow-in-ipmi_get_session_info.patch

   1 From 4f7778ed232a92bde388f38917b94f458a82c78e Mon Sep 17 00:00:00 2001
   2 From: Chrostoper Ertl <chertl@microsoft.com>
   3 Date: Thu, 28 Nov 2019 16:51:49 +0000
   4 Subject: [PATCH 08/11] session: Fix buffer overflow in ipmi_get_session_info
   5
   6 Partial fix for CVE-2020-5208, see
   7 https://github.com/ipmitool/ipmitool/security/advisories/GHSA-g659-9qxw-p7cp
   8
   9 The `ipmi_get_session_info` function does not properly check the
  10 response `data_len`, which is used as a copy size, allowing stack buffer
  11 overflow.
  12 ---
  13  lib/ipmi_session.c | 12 ++++++++----
  14  1 file changed, 8 insertions(+), 4 deletions(-)
  15
  16 diff --git a/lib/ipmi_session.c b/lib/ipmi_session.c
  17 index 141f0f4ec8dd..b9af1fd75d40 100644
  18 --- a/lib/ipmi_session.c
  19 +++ b/lib/ipmi_session.c
  20 @@ -309,8 +309,10 @@ ipmi_get_session_info(struct ipmi_intf         * intf,
  21                 }
  22                 else
  23                 {
  24 -                       memcpy(&session_info,  rsp->data, rsp->data_len);
  25 -                       print_session_info(&session_info, rsp->data_len);
  26 +                       memcpy(&session_info,  rsp->data,
  27 +                              __min(rsp->data_len, sizeof(session_info)));
  28 +                       print_session_info(&session_info,
  29 +                                          __min(rsp->data_len, sizeof(session_info)));
  30                 }
  31                 break;
  32
  33 @@ -341,8 +343,10 @@ ipmi_get_session_info(struct ipmi_intf         * intf,
  34                                 break;
  35                         }
  36
  37 -                       memcpy(&session_info,  rsp->data, rsp->data_len);
  38 -                       print_session_info(&session_info, rsp->data_len);
  39 +                       memcpy(&session_info,  rsp->data,
  40 +                              __min(rsp->data_len, sizeof(session_info)));
  41 +                       print_session_info(&session_info,
  42 +                                          __min(rsp->data_len, sizeof(session_info)));
  43
  44                 } while (i <= session_info.session_slot_count);
  45                 break;
  46 --
  47 2.27.0
  48