From c336a77283197bd63475f0773b78473c7c239567 Mon Sep 17 00:00:00 2001 From: Stephen Cprek Date: Fri, 17 Nov 2017 15:49:30 -0600 Subject: Handle comments from pre-verify and runtime lid loading commits Change-Id: I224079808493c062f04b7c3a59d45128a8f2e699 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/49875 Tested-by: Jenkins Server Reviewed-by: Michael Baiocchi Reviewed-by: Nicholas E. Bofferding Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Tested-by: FSP CI Jenkins Reviewed-by: Daniel M. Crowell --- src/usr/secureboot/runtime/test/testsecureboot_rt.H | 17 ++++++++++++----- 1 file changed, 12 insertions(+), 5 deletions(-) (limited to 'src/usr/secureboot/runtime') diff --git a/src/usr/secureboot/runtime/test/testsecureboot_rt.H b/src/usr/secureboot/runtime/test/testsecureboot_rt.H index f728357e2..380b9eb0c 100644 --- a/src/usr/secureboot/runtime/test/testsecureboot_rt.H +++ b/src/usr/secureboot/runtime/test/testsecureboot_rt.H @@ -191,11 +191,18 @@ class SecurebootRtTestSuite: public CxxTest::TestSuite } else { - TS_FAIL("testAccessSecurePnorSection: unexpected reason code for Secure Section %s. Expected RC 0x%.4X Actual RC 0x%.4X", - PNOR::SectionIdToString(i_id), - PNOR::RC_RTPNOR_INVALID_SECTION, - l_errl->reasonCode()); - errlCommit(l_errl, SECURE_COMP_ID); + if (l_errl) + { + TS_FAIL("testAccessSecurePnorSection: unexpected reason code for Secure Section %s. Expected RC 0x%.4X Actual RC 0x%.4X", + PNOR::SectionIdToString(i_id), + PNOR::RC_RTPNOR_INVALID_SECTION, + l_errl->reasonCode()); + errlCommit(l_errl, SECURE_COMP_ID); + } + else + { + TS_FAIL("testAccessSecurePnorSection: no error found when one was expected"); + } } } else if(l_errl) -- cgit v1.2.1