am b124c9e8: Fix unchecked length in Blob creation

* commit 'b124c9e86a5f8466f527501c6677b4b1b165c0b1':
  Fix unchecked length in Blob creation

1 files changed, 9 insertions, 1 deletions

diff --git a/keystore/keystore.cpp b/keystore/keystore.cpp
index 987e306..605df3b 100644
--- a/keystore/keystore.cpp
+++ b/keystore/keystore.cpp
@@ -410,8 +410,16 @@ static const uint8_t CURRENT_BLOB_VERSION = 2;
 
 class Blob {
 public:
-    Blob(const uint8_t* value, int32_t valueLength, const uint8_t* info, uint8_t infoLength,
+    Blob(const uint8_t* value, size_t valueLength, const uint8_t* info, uint8_t infoLength,
             BlobType type) {
+        if (valueLength > sizeof(mBlob.value)) {
+            valueLength = sizeof(mBlob.value);
+            ALOGW("Provided blob length too large");
+        }
+        if (infoLength + valueLength > sizeof(mBlob.value)) {
+            infoLength = sizeof(mBlob.value) - valueLength;
+            ALOGW("Provided info length too large");
+        }
         mBlob.length = valueLength;
         memcpy(mBlob.value, value, valueLength);