diff options
| author | Valentin Longchamp <valentin.longchamp@keymile.com> | 2012-08-14 01:16:36 +0000 |
|---|---|---|
| committer | Prafulla Wadaskar <prafulla@marvell.com> | 2012-09-03 17:32:58 +0530 |
| commit | 62c9b9603e2552d6b39985738a957b51b0367e72 (patch) | |
| tree | 989770589c41cf1b8538bebe41200ec124ba0f4a /board | |
| parent | 8203b201eaa1b33758956294b3ec70b326f8ba5c (diff) | |
| download | talos-obmc-uboot-62c9b9603e2552d6b39985738a957b51b0367e72.tar.gz talos-obmc-uboot-62c9b9603e2552d6b39985738a957b51b0367e72.zip | |
km/ivm: fix string len check to support 7 char board names
The fanless boards now have a 7-digit (XXXXX-F) board name. This
triggers a border condition when reading this string in the IVM although
this string is smaller than the currenly read string size, but only by 1
character.
This patch corrects this by changing the size check condition for string
length. It is the same change that was done in the platform for this
same bug.
Signed-off-by: Valentin Longchamp <valentin.longchamp@keymile.com>
cc: Holger Brunck <holger.brunck@keymile.com>
cc: Stefan Bigler <stefan.bigler@keymile.com>
Diffstat (limited to 'board')
| -rw-r--r-- | board/keymile/common/ivm.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/board/keymile/common/ivm.c b/board/keymile/common/ivm.c index 9bc3c21c9c..eaa924f0e6 100644 --- a/board/keymile/common/ivm.c +++ b/board/keymile/common/ivm.c @@ -163,7 +163,7 @@ static int ivm_findinventorystring(int type, if (addr == INVENTORYDATASIZE) { xcode = -1; printf("Error end of string not found\n"); - } else if ((size >= (maxlen - 1)) && + } else if ((size > (maxlen - 1)) && (buf[addr] != '\r')) { xcode = -1; printf("string too long till next CR\n"); |
