diff options
author | Richard J. Knight <rjknight@us.ibm.com> | 2017-06-27 12:16:16 -0500 |
---|---|---|
committer | Sachin Gupta <sgupta2m@in.ibm.com> | 2017-11-23 07:48:54 -0500 |
commit | 7f62dd04344bba5ea02d8d1d19328a6cd33bcb8c (patch) | |
tree | d81bdbe647f7dca593165465747b9b618247ec81 /src/sbefw/sbeSecureMemRegionManager.C | |
parent | 0bc318357fc86b96424414d6e52b9326d15c279a (diff) | |
download | talos-sbe-7f62dd04344bba5ea02d8d1d19328a6cd33bcb8c.tar.gz talos-sbe-7f62dd04344bba5ea02d8d1d19328a6cd33bcb8c.zip |
p9_xip_tool support for DD level section parsing
- Currently supports only .overlays section.
- Fixed an heap problem causing segmentation fault after repetitive
use of malloc in rs4_decompress() and rs4_extract_cmsk(). Now allocating
buffers locally and calling _rs4_decompress() and _rs4_extract_cmsk()
instead so only have to use malloc once.
- Fixed a bug in the usage of the ringBlockPtr which got moved away from
pointing to its initially allocated buffer after the first CMSK ring is
processed.
- Even though malloc() can be used in a C++ context, we really should be
using the new() operator. So, I replaced all malloc() and free() with
new() and delete() instead in both p9_xip_tool.C and
p9_scan_compression.C.
Change-Id: I2da7509ed7aaa13345185dc07bce57f71c3740fd
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/42531
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: HWSV CI <hwsv-ci+hostboot@us.ibm.com>
Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Richard J. Knight <rjknight@us.ibm.com>
Reviewed-by: Sumit Kumar <sumit_kumar@in.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/42802
Reviewed-by: Hostboot Team <hostboot@us.ibm.com>
Reviewed-by: Sachin Gupta <sgupta2m@in.ibm.com>
Diffstat (limited to 'src/sbefw/sbeSecureMemRegionManager.C')
0 files changed, 0 insertions, 0 deletions