Patrick Williams | feb85db | 2015-02-16 12:10:07 -0600 | [diff] [blame] | 1 | From 7a51e37d5d7672fbef3a8f3c4a11a652ded9241a Mon Sep 17 00:00:00 2001 |
| 2 | From: Andres Lugo-Reyes <aalugore@us.ibm.com> |
| 3 | Date: Mon, 16 Feb 2015 10:16:48 -0600 |
| 4 | Subject: [PATCH] Add loop fix and missing include to attribute override tool. |
| 5 | |
| 6 | Change-Id: I451d905d96822a151aa7966d31d27353bebbe953 |
| 7 | Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/15757 |
| 8 | Tested-by: Jenkins Server |
| 9 | Reviewed-by: Brian Silver <bsilver@us.ibm.com> |
| 10 | Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com> |
| 11 | --- |
| 12 | src/usr/targeting/attroverride/attrTextToBinaryBlob.C | 3 ++- |
| 13 | 1 file changed, 2 insertions(+), 1 deletion(-) |
| 14 | |
| 15 | diff --git a/src/usr/targeting/attroverride/attrTextToBinaryBlob.C b/src/usr/targeting/attroverride/attrTextToBinaryBlob.C |
| 16 | index 10ef7bb..e4b3330 100755 |
| 17 | --- a/src/usr/targeting/attroverride/attrTextToBinaryBlob.C |
| 18 | +++ b/src/usr/targeting/attroverride/attrTextToBinaryBlob.C |
| 19 | @@ -48,6 +48,7 @@ |
| 20 | #include <stdio.h> |
| 21 | #include <arpa/inet.h> |
| 22 | #include <endian.h> |
| 23 | +#include <unistd.h> |
| 24 | #include <hwpf/fapi/fapiTarget.H> |
| 25 | #include <attributeenums.H> |
| 26 | #include <pnor/ecc.H> |
| 27 | @@ -1024,7 +1025,7 @@ bool AttrTextToBinaryBlob::getAttrDataFromMap(const char * i_attrString, |
| 28 | |
| 29 | o_attrId = currentAttr->iv_attrId; |
| 30 | o_attrElemSizeBytes = currentAttr->iv_attrElemSizeBytes; |
| 31 | - for(size_t i = 0; i < sizeof(o_dims)/sizeof(uint32_t); ++i) |
| 32 | + for(size_t i = 0; i < sizeof(o_dims)/sizeof(size_t); ++i) |
| 33 | { |
| 34 | o_dims[i] = currentAttr->iv_dims[i]; |
| 35 | } |
| 36 | -- |
| 37 | 2.1.1 |
| 38 | |