Handle errors in cbor_value_advance

Fixes infinite loops on input like

  unsigned char buf[] = {0x81, 0xff, 0xA2, 0x61, 0x6E, 0x64, 0x74, 0x65, 0x73, 0x74, 0x61, 0x76, 0x7, 0x0};
  thing.decode((uint8_t*)buf, sizeof(buf));

Author: facchinm

ArduinoCloudThing.cpp

@@ -200,14 +200,20 @@ void ArduinoCloudThing::decode(uint8_t * payload, size_t length) {
         CborType type = cbor_value_get_type(&recursedMap);
         if (type != CborMapType) {
             // stop the decode when 1st item thai is not a cbor map is found.
-            cbor_value_advance(&dataArray);
+            CborError err = cbor_value_advance(&dataArray);
+            if (err != CborNoError) {
+                break;
+            }
             continue;
         } else {
 
             while (!cbor_value_at_end(&recursedMap)) {
                  // if the current element is not a cbor object as expected, skip it and go ahead.
                 if (cbor_value_get_type(&recursedMap) != CborMapType) {
-                    cbor_value_advance(&recursedMap);
+                    CborError err = cbor_value_advance(&recursedMap);
+                    if (err != CborNoError) {
+                        break;
+                    }
                     continue;
                 }