Merge pull request #5 from ACRIOS-Systems/bugfix/findUbxMessageLen_ret_wrong

Hadatko · web-flow · commit 728e5c882977 · 2024-03-04T12:14:15.000+01:00
findUbxMessageLen was returning wrong length in case startIdx!=0
diff --git a/lang/cpp/src/parseUbxMessageBase.cpp b/lang/cpp/src/parseUbxMessageBase.cpp
@@ -27,7 +27,7 @@ int32_t ParseUbxMessageBase::findUbxMessageLen(const uint8_t *buf, uint16_t len,
     {
         if ((startIdx + IDX_LEN_H + 1U) <= len)
         {
-            ret = startIdx + SerializeCommon::UbxControlBytesSize + buf[startIdx + IDX_LEN_L] + (buf[startIdx + IDX_LEN_H] << 8U);
+            ret = SerializeCommon::UbxControlBytesSize + buf[startIdx + IDX_LEN_L] + (buf[startIdx + IDX_LEN_H] << 8U);
         }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ int32_t ParseUbxMessageBase::findUbxMessageLen(const uint8_t *buf, uint16_t len,`
`27`	`27`	`{`
`28`	`28`	`if ((startIdx + IDX_LEN_H + 1U) <= len)`
`29`	`29`	`{`
`30`		`- ret = startIdx + SerializeCommon::UbxControlBytesSize + buf[startIdx + IDX_LEN_L] + (buf[startIdx + IDX_LEN_H] << 8U);`
	`30`	`+ ret = SerializeCommon::UbxControlBytesSize + buf[startIdx + IDX_LEN_L] + (buf[startIdx + IDX_LEN_H] << 8U);`
`31`	`31`	`}`
`32`	`32`	`}`
`33`	`33`