Fixed tests

Author: rjtokenring

extras/test/src/test_command_decode.cpp

@@ -123,7 +123,7 @@ SCENARIO("Test the decoding of command messages") {
       REQUIRE(genericMessage->id == (uint32_t)67072);
     }
 
-    delete msg->fields.params.last_values;
+    free(msg->fields.params.last_values);
     delete msg;
   }
 
@@ -160,6 +160,24 @@ SCENARIO("Test the decoding of command messages") {
       REQUIRE(err == MessageDecoder::DecoderState::Success);
       REQUIRE(strcmp(msg->fields.params.id, otaIdToMatch) == 0);
       REQUIRE(strcmp(msg->fields.params.url, urlToMatch) == 0);
+      // Initial SHA256 check
+      REQUIRE(msg->fields.params.initialSha256[0] == (uint8_t)0x33);
+      REQUIRE(msg->fields.params.initialSha256[1] == (uint8_t)0x30);
+      REQUIRE(msg->fields.params.initialSha256[2] == (uint8_t)0x30);
+      REQUIRE(msg->fields.params.initialSha256[3] == (uint8_t)0x34);
+      REQUIRE(msg->fields.params.initialSha256[4] == (uint8_t)0x61);
+      REQUIRE(msg->fields.params.initialSha256[5] == (uint8_t)0x61);
+      REQUIRE(msg->fields.params.initialSha256[6] == (uint8_t)0x62);
+      REQUIRE(msg->fields.params.initialSha256[7] == (uint8_t)0x32);
+
+      // Final SHA256 check
+      REQUIRE(msg->fields.params.finalSha256[0] == (uint8_t)0x6A);
+      REQUIRE(msg->fields.params.finalSha256[1] == (uint8_t)0x6B);
+      REQUIRE(msg->fields.params.finalSha256[2] == (uint8_t)0x61);
+      REQUIRE(msg->fields.params.finalSha256[3] == (uint8_t)0x73);
+      REQUIRE(msg->fields.params.finalSha256[4] == (uint8_t)0x64);
+      REQUIRE(msg->fields.params.finalSha256[5] == (uint8_t)0x6B);
+
       REQUIRE(genericMessage->id == (uint32_t)65792);
     }
 

src/cbor/MessageDecoder.cpp

@@ -99,7 +99,7 @@ bool copyCBORByteToArray(CborValue * param, uint8_t * dest, size_t dest_size) {
         if (val_size > dest_size) {
           val_size = dest_size;
         }
-        std::copy(val, val + dest_size, dest);
+        std::copy(val, val + val_size, dest);
         free(val);
         return true;
       }