Add patch from Debian to fix tsol buffer overflow

Originally reported by Jan Safranek in
https://sourceforge.net/p/ipmitool/mailman/message/24405281/
but never applied upstream.

Problem seen and reported in real usage:
https://bugs.launchpad.net/ubuntu/+source/ipmitool/+bug/633054
This commit is contained in:
Pavel Cahyna 2024-02-12 19:39:16 +01:00
parent 9c62e14a9a
commit 8d70a87b63
2 changed files with 27 additions and 0 deletions

View File

@ -0,0 +1,22 @@
Description: fix buffer overflow
based on 101_fix_buf_overflow from Leo Iannacone <l3on@ubuntu.com>
Author: Jörg Frings-Fürst <debian@jff-webhosting.net>
Bug: TSOL buffer overflow
Bug-ubuntu: https://bugs.launchpad.net/ubuntu/+source/ipmitool/+bug/633054
Forwarded: https://sourceforge.net/p/ipmitool/patches/100/
Last-Update: 2014-12-01
---
This patch header follows DEP-3: http://dep.debian.net/deps/dep3/
Index: trunk/lib/ipmi_tsol.c
===================================================================
--- trunk.orig/lib/ipmi_tsol.c
+++ trunk/lib/ipmi_tsol.c
@@ -374,7 +374,7 @@ ipmi_tsol_main(struct ipmi_intf *intf, i
char *recvip = NULL;
char in_buff[IPMI_BUF_SIZE];
char out_buff[IPMI_BUF_SIZE * 8];
- char buff[IPMI_BUF_SIZE + 4];
+ char buff[IPMI_BUF_SIZE * 8 + 4];
int fd_socket, result, i;
size_t out_buff_fill, in_buff_fill;
int ip1, ip2, ip3, ip4;

View File

@ -26,6 +26,11 @@ Patch7: 0007-check-input.patch
# https://github.com/ipmitool/ipmitool/pull/214 - approved but not merged # https://github.com/ipmitool/ipmitool/pull/214 - approved but not merged
Patch14: 0014-lanplus-cipher-retry.patch Patch14: 0014-lanplus-cipher-retry.patch
# Debian patches, never applied upstream
# https://bugs.launchpad.net/ubuntu/+source/ipmitool/+bug/633054
# https://sourceforge.net/p/ipmitool/mailman/message/24405281/
Patch100: 0100-fix_buf_overflow.patch
BuildRequires: openssl-devel readline-devel ncurses-devel BuildRequires: openssl-devel readline-devel ncurses-devel
%{?systemd_requires} %{?systemd_requires}
BuildRequires: systemd BuildRequires: systemd