diff options
author | Mike Jones <mjjones@us.ibm.com> | 2013-01-03 14:22:51 -0600 |
---|---|---|
committer | A. Patrick Williams III <iawillia@us.ibm.com> | 2013-01-04 14:55:43 -0600 |
commit | c2192639ff1e74f739d89420e0d7fbd5bf1b602b (patch) | |
tree | 288db2eb7d77de5a9e066fe45edbc975682ffc96 /src/usr/hwpf/plat | |
parent | fb47cd30d0c8bfedd302aabee0ee974e147738ad (diff) | |
download | talos-hostboot-c2192639ff1e74f739d89420e0d7fbd5bf1b602b.tar.gz talos-hostboot-c2192639ff1e74f739d89420e0d7fbd5bf1b602b.zip |
HWPF: Attribute Override. Bugfix checking num attrs in message
Change-Id: I86167f82c45a1fdbf26c1be3e6cf28432ca76327
Reviewed-on: http://gfw160.austin.ibm.com:8080/gerrit/2807
Reviewed-by: Brian H. Horton <brianh@linux.ibm.com>
Tested-by: Jenkins Server
Reviewed-by: Van H. Lee <vanlee@us.ibm.com>
Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
Diffstat (limited to 'src/usr/hwpf/plat')
-rw-r--r-- | src/usr/hwpf/plat/fapiPlatAttrOverrideSync.C | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/src/usr/hwpf/plat/fapiPlatAttrOverrideSync.C b/src/usr/hwpf/plat/fapiPlatAttrOverrideSync.C index baede954b..baadaec83 100644 --- a/src/usr/hwpf/plat/fapiPlatAttrOverrideSync.C +++ b/src/usr/hwpf/plat/fapiPlatAttrOverrideSync.C @@ -5,7 +5,7 @@ /* */ /* IBM CONFIDENTIAL */ /* */ -/* COPYRIGHT International Business Machines Corp. 2012 */ +/* COPYRIGHT International Business Machines Corp. 2012,2013 */ /* */ /* p1 */ /* */ @@ -103,12 +103,13 @@ void monitorForFspMessages() if (l_pMsg->type == MSG_SET_OVERRIDES) { // FSP is setting attribute override(s). - FAPI_INF("monitorForFspMessages: MSG_SET_OVERRIDES"); uint64_t l_size = l_pMsg->data[1]; + FAPI_INF("monitorForFspMessages: MSG_SET_OVERRIDES (%lld overrides)", + l_size / sizeof(fapi::Attribute)); Attribute * l_pAttribute = reinterpret_cast<Attribute *>(l_pMsg->extra_data); - while (l_size > sizeof(fapi::Attribute)) + while (l_size >= sizeof(fapi::Attribute)) { Singleton<fapi::OverrideAttributeTank>::instance(). setAttribute(*l_pAttribute); |