Skip to content

Commit e6e1cf5

Browse files
sannerddcrowell77
authored andcommitted
Fix crash in err manager RP when non acked errors in PNOR
Change-Id: If2a46c6311765fa5e86b4e2e211aa10e1fd25cbf Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/36522 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Reviewed-by: Prachi Gupta <pragupta@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
1 parent 6782606 commit e6e1cf5

File tree

1 file changed

+6
-7
lines changed

1 file changed

+6
-7
lines changed

src/usr/errl/errlmanager_common.C

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
/* */
66
/* OpenPOWER HostBoot Project */
77
/* */
8-
/* Contributors Listed Below - COPYRIGHT 2015,2016 */
8+
/* Contributors Listed Below - COPYRIGHT 2015,2017 */
99
/* [+] International Business Machines Corp. */
1010
/* */
1111
/* */
@@ -622,13 +622,12 @@ void ErrlManager::sendErrLogToBmc(errlHndl_t &io_err, bool i_sendSels)
622622
if(!i_sendSels)
623623
{
624624
const char* l_prev_boot = "Error from a previous boot";
625-
ERRORLOG::ErrlUserDetailsString(l_prev_boot).addToLog(io_err);
625+
// Create a raw user-defined section. It does a copy of the
626+
// string constant, then once added to io_err, io_err owns mem
627+
ErrlUD* l_ffdcSection = new ErrlUD(l_prev_boot, strlen(l_prev_boot),
628+
ERRL_COMP_ID, 1, ERRL_UDT_STRING );
626629
io_err->iv_SectionVector.insert(io_err->iv_SectionVector.begin(),
627-
1,io_err->iv_SectionVector.back());
628-
if(io_err->iv_SectionVector.size()!=0)
629-
{
630-
io_err->iv_SectionVector.pop_back();
631-
}
630+
l_ffdcSection);
632631
}
633632

634633
// flatten into buffer, truncate to max eSEL size

0 commit comments

Comments
 (0)