commit 57dd144a3d92190ed51dcffb1a7a108469604240
author Laurence Lundblade <lgl@island-resort.com> Mon Oct 15 20:26:28 2018 +0530
committer Laurence Lundblade <lgl@island-resort.com> Mon Oct 15 20:26:28 2018 +0530
tree 54a637b3dd2f51752aa88abe8f8a0f59f78bedb7
parent 6de3706e62a2ade1a1411093fa304e0e0cb9685b

bug fix for indefinite string labels; lots of tests running and passing for indefinite strings; seems to be complete

src/qcbor_decode.c

diff --git a/src/qcbor_decode.c b/src/qcbor_decode.c
index 676ec32..6b89df6 100644
--- a/src/qcbor_decode.c
+++ b/src/qcbor_decode.c
@@ -586,7 +586,7 @@
    
    pDecodedItem->uTagBits = 0;
    pDecodedItem->uTag     = 0;
-   pDecodedItem->uAllocated = 0;
+   pDecodedItem->uDataAlloc = 0;
    
    // At this point the major type and the value are valid. We've got the type and the number that
    // starts every CBOR data item.
@@ -694,7 +694,7 @@
       if(Item.uDataType == QCBOR_TYPE_BREAK) {
          // String is complete
          pDecodedItem->val.string = FullString;
-         pDecodedItem->uAllocated = 1;
+         pDecodedItem->uDataAlloc = 1;
          break;
       }
       
@@ -841,7 +841,7 @@
       
       // Get the next item which will be the real data; Item will be the label
       QCBORItem LabelItem = *pDecodedItem;
-      nReturn = GetNext_TaggedItem(me, pDecodedItem); // TODO: test indefinite length string labels
+      nReturn = GetNext_TaggedItem(me, pDecodedItem);
       if(nReturn)
          goto Done;
       
@@ -861,6 +861,7 @@
          pDecodedItem->uLabelType = QCBOR_TYPE_UINT64;
       } else if(LabelItem.uDataType == QCBOR_TYPE_BYTE_STRING) {
          pDecodedItem->label.string = LabelItem.val.string;
+         pDecodedItem->uLabelAlloc = LabelItem.uDataAlloc;
          pDecodedItem->uLabelType = QCBOR_TYPE_BYTE_STRING;
       } else {
          // label is not an int or a string. It is an arrray