diff options
| author | Matt Spinler <spinler@us.ibm.com> | 2019-10-09 14:07:52 -0500 |
|---|---|---|
| committer | Matt Spinler <spinler@us.ibm.com> | 2019-10-22 14:09:56 +0000 |
| commit | 31eed996da81cf6e35e7ddd035aa8088be230218 (patch) | |
| tree | d23523953e068e0e77ab2cb392584743f2d17be5 /extensions/openpower-pels/private_header.cpp | |
| parent | f9bae18539eca100632d462083698b57d9687c82 (diff) | |
| download | phosphor-logging-31eed996da81cf6e35e7ddd035aa8088be230218.tar.gz phosphor-logging-31eed996da81cf6e35e7ddd035aa8088be230218.zip | |
PEL: Initialize PrivateHeader fields in ctor
In the stream constructor, initialize the member variables to 0 before
unflattening.
This only matters when the data is so short that the 'section count'
field doesn't get initialized and so the code tries to loop through and
unflatten that many optional sections, and since there aren't actually
any it was spitting out extra journal traces about invalid section IDs.
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: If08e9ae244d5ade96cafa2160ad9714bebe34551
Diffstat (limited to 'extensions/openpower-pels/private_header.cpp')
| -rw-r--r-- | extensions/openpower-pels/private_header.cpp | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/extensions/openpower-pels/private_header.cpp b/extensions/openpower-pels/private_header.cpp index 2265497..64b1f2c 100644 --- a/extensions/openpower-pels/private_header.cpp +++ b/extensions/openpower-pels/private_header.cpp @@ -48,7 +48,9 @@ PrivateHeader::PrivateHeader(uint16_t componentID, uint32_t obmcLogID, _valid = true; } -PrivateHeader::PrivateHeader(Stream& pel) +PrivateHeader::PrivateHeader(Stream& pel) : + _creatorID(0), _logType(0), _reservedByte(0), _sectionCount(0), + _obmcLogID(0), _plid(0), _id(0) { try { |
