Package: ipmitool / 1.8.19-10

Metadata

Package Version Patches format
ipmitool 1.8.19-10 3.0 (quilt)

Patch series

view the series file
Patch File delta Description
0105 sensor_reading.patch | (download)

lib/ipmi_sdr.c | 2 1 + 1 - 0 !
lib/ipmi_sensor.c | 2 1 + 1 - 0 !
2 files changed, 2 insertions(+), 2 deletions(-)

 fix soensor reading
0620 manpage_typo.patch | (download)

doc/ipmievd.8.in | 2 1 + 1 - 0 !
doc/ipmitool.1.in | 86 43 + 43 - 0 !
2 files changed, 44 insertions(+), 44 deletions(-)

 fix man page typos
0700 build.patch | (download)

Makefile.am | 38 19 + 19 - 0 !
configure.ac | 30 15 + 15 - 0 !
2 files changed, 34 insertions(+), 34 deletions(-)

 remove downloads at build time
0701 disable libtool silent.patch | (download)

configure.ac | 6 2 + 4 - 0 !
1 file changed, 2 insertions(+), 4 deletions(-)

 enable verbose output of libtool and fix warning
0100 fix_buf_overflow.patch | (download)

lib/ipmi_tsol.c | 2 1 + 1 - 0 !
1 file changed, 1 insertion(+), 1 deletion(-)

 fix buffer overflow
 based on 101_fix_buf_overflow from Leo Iannacone <l3on@ubuntu.com> 
0010 utf8.patch | (download)

AUTHORS | 2 1 + 1 - 0 !
1 file changed, 1 insertion(+), 1 deletion(-)

 conv authors to utf-8
0110 unpdate_IANA_URL.patch | (download)

Makefile.am | 2 1 + 1 - 0 !
doc/ipmievd.8.in | 6 3 + 3 - 0 !
doc/ipmitool.1.in | 6 3 + 3 - 0 !
lib/ipmi_strings.c | 2 1 + 1 - 0 !
4 files changed, 8 insertions(+), 8 deletions(-)

 fix iana url
0115 hurd_PATH_MAX.patch | (download)

include/ipmitool/ipmi_strings.h | 4 4 + 0 - 0 !
1 file changed, 4 insertions(+)

 add missing path_max for hurd-i386
0625 manpage_wrong_time_set.patch | (download)

doc/ipmitool.1.in | 6 3 + 3 - 0 !
1 file changed, 3 insertions(+), 3 deletions(-)

 fix description of "sel time set"
0800 fix buffer overrun.patch | (download)

src/plugins/open/open.c | 2 1 + 1 - 0 !
1 file changed, 1 insertion(+), 1 deletion(-)

 [patch] open: eliminate buffer overrun (#24)

clangd reports a buffer overrun issue in `open` interface.

The sprintf() used to fill ipmi_devfs2 requires 17 bytes to store the
null terminated string. The character buffer is only 16 bytes in
length.

Signed-off-by: Johnathan Mantey <johnathanx.mantey@intel.com>
Reviewed-on: https://codeberg.org/IPMITool/ipmitool/pulls/24
0801 fix lan print fails on unsupported parameters.patch | (download)

lib/ipmi_lanp.c | 2 1 + 1 - 0 !
1 file changed, 1 insertion(+), 1 deletion(-)

 [patch] lan: fix lan print fails on unsupported parameters

After upgrading to ipmitool 1.8.19, ipmitool lan print can only print out
`Set in Progress` and other parameters are missing on our servers. After
bisecting, commit:

  351dad24a26f lan: Add processing of get/set specific CCs

is identified to be the source of the problem, where the function
get_lan_param_select is expected to consider severial error codes it
receives as empty response. It then constructs an empty response in `p`
and assigns `p` to `rc` and the control flow falls through to the
default case, which prints the error code in verbose mode and should
return `rc` instead of `NULL`.

Signed-off-by: Miao Wang <shankerwangmiao@gmail.com>
0802 fix temp read parameters.patch | (download)

lib/ipmi_dcmi.c | 4 2 + 2 - 0 !
1 file changed, 2 insertions(+), 2 deletions(-)

 [patch] dcmi: update parameters to read temperature sensors

Issue:
When the system has number of CPU temperature sensors more than 8,
"ipmitool dcmi get_temp_reading" can not show all of sensors.

Root cause:
To request to read multiple sensors for each message, ipmitool has to
send "Get Temperature Readings" command with the "Entity Instance" always
0 and the "Entity Instance Start" as the offset. But the current code is
sending "Entity Instance" is offset and "Entity Instance Start" always is
0. It makes ipmitool only read 1 sensor each time. Besides that, the
"Entity Instance Start" value starts from 1 (not 0), therefore, the
initialization has to be set to 1.

Solution:
This commit corrects the order of parameters and the initialization of
"Entity Instance Start" byte.

Resolves ipmitool/ipmitool#3

Tested:
1. Update BMC software to support 24 CPU temperature sensors
2. Request to read the temperature sensors
$ipmitool dcmi get_temp_reading
3. Display full 24 CPU temperature sensors.

Signed-off-by: Thang Tran <thuutran@amperecomputing.com>
0803 fix sdr hex value.patch | (download)

lib/ipmi_sdr.c | 15 8 + 7 - 0 !
1 file changed, 8 insertions(+), 7 deletions(-)

 [patch] sdr: refix 6e037d6bfbbb93b349c8ca331ebde03a (#41)

A bug was introduced by commit 6e037d6bfbbb93b349c8ca331ebde03a837f76bf
due to which the command `ipmitool sdr type` stopped accepting raw
hex values for the type and would only accept strings.

Fix that by partially reverting the troublesome commit.

Additionally, apply the logic of that commit to calls of
`strcasecmp()` in ipmi_sdr.c.

Resolves https://codeberg.org/IPMITool/ipmitool/issues/41

Signed-off-by: Alexander Amelkin <alexander@amelkin.msk.ru>
0130 chassis_bootparam.patch | (download)

lib/ipmi_chassis.c | 2 1 + 1 - 0 !
1 file changed, 1 insertion(+), 1 deletion(-)

 fix 'chassis bootparam .. options=no-timeout'