diff options
author | Dean Sanner <dsanner@us.ibm.com> | 2017-10-21 22:05:10 -0500 |
---|---|---|
committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2017-11-01 15:39:55 -0400 |
commit | 5b272bf3f2322a16d2d0577066f6894b81f3f1b0 (patch) | |
tree | debcb77c6a5d34639fb29ebd937fe206afaaefc4 /src/usr/util | |
parent | aa313fb234e1d00c425e2df84b59725f318d6548 (diff) | |
download | talos-hostboot-5b272bf3f2322a16d2d0577066f6894b81f3f1b0.tar.gz talos-hostboot-5b272bf3f2322a16d2d0577066f6894b81f3f1b0.zip |
Fix PNOR sections loaded into reserve-mem for non secure sys
Prevent the header page from being loaded into HB reserved memory
for unsigned PNOR sections with a header.
Change-Id: I0fe80edc1a171b3e3066348cbf64205041082e2b
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/48760
Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Reviewed-by: Dean Sanner <dsanner@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Diffstat (limited to 'src/usr/util')
-rw-r--r-- | src/usr/util/runtime/utillidmgr_rt.C | 15 |
1 files changed, 3 insertions, 12 deletions
diff --git a/src/usr/util/runtime/utillidmgr_rt.C b/src/usr/util/runtime/utillidmgr_rt.C index 338f22b87..52490cac1 100644 --- a/src/usr/util/runtime/utillidmgr_rt.C +++ b/src/usr/util/runtime/utillidmgr_rt.C @@ -178,20 +178,11 @@ errlHndl_t UtilLidMgr::loadLid() else { // If no secure header, just use PNOR size + // NOTE: Unsigned sections with sha512Version have already + // skipped over the header and decreased size while + // parsing the PNOR TOC iv_lidSize = iv_lidPnorInfo.size; UTIL_FT("UtilLidMgr::loadLid - iv_lidSize=%d", iv_lidSize ); - - // Need to increment past header if one exists - if (iv_lidPnorInfo.sha512Version) - { - UTIL_FT("UtilLidMgr::loadLid - resv mem section is not secure, but has header"); - // Increment by page size to not expose sha512 version header - iv_lidBuffer = static_cast<uint8_t*>(iv_lidBuffer) + - PAGESIZE; - iv_lidSize -= PAGESIZE; - UTIL_FT("UtilLidMgr::loadLid - iv_lidSize=%d", - iv_lidSize ); - } } } } |