commit 70b17f9353cdb025f76fcec543d12e29514b3257
author Patrick Venture <venture@google.com> Mon Oct 28 20:01:53 2019 -0700
committer Patrick Venture <venture@google.com> Tue Oct 29 19:53:41 2019 +0000
tree 684eb224136541921de1351c6178482f29a94752
parent b57098a6824ed5aad4a8d7e8964492351388b882

bugfix: storagecommands: read fru: return bytes included

The requested number of bytes could exceed the bytes remaining,
therefore only return the length of the bytes returned.

Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: Id1e107b07fc787bf49d0b0455be2e4e10750b798

src/storagecommands.cpp

diff --git a/src/storagecommands.cpp b/src/storagecommands.cpp
index 1d8763c..7b41d8d 100644
--- a/src/storagecommands.cpp
+++ b/src/storagecommands.cpp
@@ -314,7 +314,8 @@
         requestedData.begin(), fruCache.begin() + fruInventoryOffset,
         fruCache.begin() + fruInventoryOffset + fromFruByteLen);
 
-    return ipmi::responseSuccess(countToRead, requestedData);
+    return ipmi::responseSuccess(static_cast<uint8_t>(requestedData.size()),
+                                 requestedData);
 }
 
 /** @brief implements the write FRU data command