ipmitool delloem: ipmitool delloem extension always return success - fix it

The delloem extension functions all return sane error codes, but they
are not used and not passed on. Fix it.

Signed-off-

ipmitool delloem: ipmitool delloem extension always return success - fix it

The delloem extension functions all return sane error codes, but they
are not used and not passed on. Fix it.

Signed-off-by: Thomas Renninger <trenn@suse.de>

show more ...

ID: 318 - ipmi_tsol.c: fix buffer overflow

Commit fixes buffer over-flow in ipmi_tsol caused by mis-calculation in buffer
size, resp. using wrong variables completely.

ID: 306 - "fru print" command prints the FRU #0 twice

This patch make the command to spit FRU info #0 only once.

Commit for Dmitry Bazhenov

ID: 305 - HPM.1 deferred activation support fixup

Currently, the HPM.1 agent in the IPMItool does not differ between Rollback
support and Deferred Activation Support. The patch fixes up this bug.

C

ID: 305 - HPM.1 deferred activation support fixup

Currently, the HPM.1 agent in the IPMItool does not differ between Rollback
support and Deferred Activation Support. The patch fixes up this bug.

Commit for Dmitry Bazhenov

show more ...

lib/ipmi_tsol.c - remove dead code

lib/ipmi_tsol.c - make code formatting saner

ID: 317 - ipmi_fwum.c: fix typo

ID: 315 - buildsystem: configure.in is deprecated

As per https://lists.gnu.org/archive/html/automake/2013-05/msg00049.html,
renamed to configure.ac

ID: 316 - Directory debian is outdated

Debian upstream will take care of packaging.

ID: 103 - 'lib/ipmi_ekanalyzer.c' needs a re-work

Most of the code is reformatted. I've removed some if blocks which didn't make
sense along the way. However, huge pile of work remains.

ID: 103 - 'lib/ipmi_ekanalyzer.c' needs a re-work

Remove trailing empty lines.

ID: 46 - SEL OEM record corner case

According to 31.6.1 SEL Record Type Ranges, IPMIv2.0 specification, timestamped
OEM SEL records are in interval of <C0h,DFh> and not <C0h,DFh).

Changes Integrated since 1.8.14RC2

ID: 311 man page update for new sunoem commands

ID: 280 - man page cleanup

Make comment about 'source offset' explicit

ID: 308 - "fru edit" no longer works for non-zero fields

1. Warnings about "FRU Area Length" based on uninitialized (malloc'd) memory
contents (due to fru->max_read_size not being initialized, left

ID: 308 - "fru edit" no longer works for non-zero fields

1. Warnings about "FRU Area Length" based on uninitialized (malloc'd) memory
contents (due to fru->max_read_size not being initialized, left at 0) and
fru_data not being zeroed after malloc() in ipmi_fru_set_field_string().

2. "fru edit" commands for any field index other than 0 would fail (with "Field
not found !" error) due to a couple offset and length calculation errors (for
all the supported "area" types) in ipmi_fru_set_field_string().

3. "fru edit" commands would corrupt the FRU Inventory Area due to incorrect
"source offset" value being specified in write_fru_area() call in
impi_fru_set_field_string().

Commit for Rob Swindell

show more ...

ID: 310 - lanplus retry regression

The lanplus driver for has a bug in ipmi_lanplus_open_session(). There is
an extra and undesired check for a null response (timeout). As a result, it
returns 1 to

ID: 310 - lanplus retry regression

The lanplus driver for has a bug in ipmi_lanplus_open_session(). There is
an extra and undesired check for a null response (timeout). As a result, it
returns 1 to the caller. The result of this is an occasional assertion. Commit
mends this.

Commit for Pat Donlin

show more ...

Add ticket IDs, if we have one

Add hpm2.h to list of distributed include files

Update ChangeLog for 1.8.14RC2 changes

Incorrect byteswap in SOL maximum payload code

max_inbound_payload_size, max_outbound_payload_size and port are
assembled byte by byte using shifts. This works correctly in both
little and big endia

Incorrect byteswap in SOL maximum payload code

max_inbound_payload_size, max_outbound_payload_size and port are
assembled byte by byte using shifts. This works correctly in both
little and big endian and doing a subsequent byte swap is wrong.

To highlight this issue I dumped the values on a big endian machine:

max_inbound_payload_size 51200
max_outbound_payload_size 51200
port 28418

And after this fix:

max_inbound_payload_size 200
max_outbound_payload_size 200
port 623

Signed-off-by: Zdenek Styblik <stybla@turnovfree.net>

show more ...

Fix build error in HPM.2 code

lib/hpm2.c uses BSWAP_16 without including ipmitool/bswap.h.

Signed-off-by: Zdenek Styblik <stybla@turnovfree.net>

ID: 300 - new sunoem functionality

Main changes include:

* direct connection to ILOM command line interface
* ability to add/delete ssh keys fot ILOM users
* ability to set ILOM properties
* ab

ID: 300 - new sunoem functionality

Main changes include:

* direct connection to ILOM command line interface
* ability to add/delete ssh keys fot ILOM users
* ability to set ILOM properties
* ability to retrieve various logs from SP
* removal of obsolete/non-functioning code and other misc changes

Commit for Martin Hovorka of Oracle

show more ...

ID: 144 - Fix 'dcmi power set_limit action <value>'

From the ticket:
(v1.8.12) "dcmi power set_limit action <value>" (where <value> is either
"sel_logging" or "power_off") does not actually send the

ID: 144 - Fix 'dcmi power set_limit action <value>'

From the ticket:
(v1.8.12) "dcmi power set_limit action <value>" (where <value> is either
"sel_logging" or "power_off") does not actually send the "Set Power Limit"
command (so no change to the power limit policy is made).

This was caused by all atoi(), strtol(), strtoul() calls being replaced. This
patch should fix it.

Commit for Holger Liebig

show more ...