diff options
| author | Dean Sanner <dsanner@us.ibm.com> | 2017-02-02 06:31:18 -0600 |
|---|---|---|
| committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2017-02-03 10:03:35 -0500 |
| commit | 47278f0f5630f7a9b001f193b76b234b9c13e26a (patch) | |
| tree | 1ba76a70c27fb43886267fbf9e279732e71fc31c /src/usr/vpd/ipvpd.C | |
| parent | b8da45026ffe3597634bdf01ed444b9106ee3f31 (diff) | |
| download | talos-hostboot-47278f0f5630f7a9b001f193b76b234b9c13e26a.tar.gz talos-hostboot-47278f0f5630f7a9b001f193b76b234b9c13e26a.zip | |
Fix VPD record size calculation
Algorithm was not including the 2 byte size field as part of
the full record length, resulting in shortened data being
returned.
Change-Id: I49a934a24771245fc7b808a2986da622b4aac4a5
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/35799
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
Reviewed-by: Prachi Gupta <pragupta@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Diffstat (limited to 'src/usr/vpd/ipvpd.C')
| -rw-r--r-- | src/usr/vpd/ipvpd.C | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/usr/vpd/ipvpd.C b/src/usr/vpd/ipvpd.C index 47fdd21a5..6c4ff9953 100644 --- a/src/usr/vpd/ipvpd.C +++ b/src/usr/vpd/ipvpd.C @@ -5,7 +5,7 @@ /* */ /* OpenPOWER HostBoot Project */ /* */ -/* Contributors Listed Below - COPYRIGHT 2013,2016 */ +/* Contributors Listed Below - COPYRIGHT 2013,2017 */ /* [+] Google Inc. */ /* [+] International Business Machines Corp. */ /* */ @@ -1533,6 +1533,7 @@ errlHndl_t IpVpdFacade::retrieveRecord( const char * i_recordName, //byteswap l_size = le16toh(l_size); + l_size += 2; // include the 2 byte size field in total record // If the buffer is NULL, return the keyword size in io_buflen if( NULL == io_buffer ) |
