update signature length

Author: fabik111

extras/test/src/test_command_encode.cpp

@@ -442,25 +442,52 @@ SCENARIO("Test the encoding of command messages") {
   {
     ProvisioningSignatureMessage command;
     command.c.id = CommandId::ProvisioningSignature;
-    memset(command.params.signature, 0xCA, 32);
+    memset(command.params.signature, 0xCA, 246);
     uint8_t buffer[512];
     size_t bytes_encoded = sizeof(buffer);
 
     CBORMessageEncoder encoder;
     Encoder::Status err = encoder.encode((Message*)&command, buffer, bytes_encoded);
 
     uint8_t expected_result[] = {
-      0xda, 0x00, 0x01, 0x20, 0x11, 0x81, 0x58, 0x20, 
+      0xda, 0x00, 0x01, 0x20, 0x11, 0x81, 0x58, 0xF6, 
       0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA,
       0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA,
       0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA,
       0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA,
+      0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA,
+      0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA,
+      0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA,
+      0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA,
+      0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA,
+      0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA,
+      0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA,
+      0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA,
+      0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA,
+      0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA,
+      0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA,
+      0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA,
+      0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA,
+      0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA,
+      0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA,
+      0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA,
+      0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA,
+      0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA,
+      0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA,
+      0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA,
+      0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA,
+      0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA,
+      0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA,
+      0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA,
+      0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA,
+      0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA,
+      0xCA, 0xCA, 0xCA, 0xCA, 0xCA, 0xCA,
     };
 
     // Test the encoding is
     //DA 00012011          # tag(73744)
     //  81                 # array(1)
-    //    58 20            # bytes(32)
+    //    58 F6            # bytes(32)
     //      CA.... omissis # values
     THEN("The encoding is successful") {
       REQUIRE(err == Encoder::Status::Complete);

src/cbor/MessageEncoder.cpp

@@ -222,13 +222,13 @@ CborError CBORMessageEncoder::encodeProvisioningListWifiNetworks(CborEncoder * a
 CborError CBORMessageEncoder::encodeProvisioningUniqueId(CborEncoder * array_encoder, Message * message)
 {
   ProvisioningUniqueIdMessage * provisioningUniqueId = (ProvisioningUniqueIdMessage *) message;
-  CHECK_CBOR(cbor_encode_byte_string(array_encoder, (uint8_t *) provisioningUniqueId->params.uniqueId, 32));
+  CHECK_CBOR(cbor_encode_byte_string(array_encoder, (uint8_t *) provisioningUniqueId->params.uniqueId, UID_SIZE));
   return CborNoError;
 }
 
 CborError CBORMessageEncoder::encodeProvisioningSignature(CborEncoder * array_encoder, Message * message)
 {
   ProvisioningSignatureMessage * provisioningSignature = (ProvisioningSignatureMessage *) message;
-  CHECK_CBOR(cbor_encode_byte_string(array_encoder, (uint8_t *) provisioningSignature->params.signature, 32));
+  CHECK_CBOR(cbor_encode_byte_string(array_encoder, (uint8_t *) provisioningSignature->params.signature, SIGNATURE_SIZE));
   return CborNoError;
 }

src/message/Commands.h

@@ -26,6 +26,8 @@
 #define URL_SIZE                   256
 #define ID_SIZE                     16
 #define MAX_LIB_VERSION_SIZE        10
+#define UID_SIZE                    32
+#define SIGNATURE_SIZE              246
 
 /******************************************************************************
     TYPEDEF
@@ -193,7 +195,7 @@ struct ProvisioningUniqueIdMessage {
 struct ProvisioningSignatureMessage {
   Command c;
   struct {
-    char signature[32]; //The payload is an array of char with a maximum length of 32, not null terminated. It's not a string.
+    char signature[246]; //The payload is an array of char with a maximum length of 246, not null terminated. It's not a string.
   } params;
 };