fixup! tlvf: swap returned value of getNextTlvLength()

Method return type is uint16_t so return UINT16_MAX instead of -1 in case of error.
prplfoundation · Mar 16, 2020 · 20eee01 · 20eee01
1 parent 3bfdde0
commit 20eee01
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/framework/tlvf/src/src/CmduMessageRx.cpp b/framework/tlvf/src/src/CmduMessageRx.cpp
@@ -61,8 +61,9 @@ int CmduMessageRx::getNextTlvType() const
 
 uint16_t CmduMessageRx::getNextTlvLength() const
 {
-    if (!getCmduHeader())
-        return -1;
+    if (!getCmduHeader()) {
+        return UINT16_MAX;
+    }
     sTlvHeader *tlv = reinterpret_cast<sTlvHeader *>(msg.prevClass()->getBuffPtr());
 
     uint16_t tlv_length = tlv->length;