Removed warning (#4693)

* Removed warning Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com> * Added suggestions Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com> --------- Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com> (cherry picked from commit 8258507)
eProsima · Apr 18, 2024 · a1b0a62 · a1b0a62
1 parent 8ce86b9
commit a1b0a62
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/cpp/fastdds/core/policy/ParameterList.cpp b/src/cpp/fastdds/core/policy/ParameterList.cpp
@@ -118,14 +118,14 @@ bool ParameterList::read_guid_from_cdr_msg(
         fastrtps::rtps::GUID_t& guid)
 {
     bool valid = false;
-    uint16_t pid;
-    uint16_t plength;
+    uint16_t pid = 0;
+    uint16_t plength = 0;
     while (msg.pos < msg.length)
     {
         valid = true;
         valid &= fastrtps::rtps::CDRMessage::readUInt16(&msg, &pid);
         valid &= fastrtps::rtps::CDRMessage::readUInt16(&msg, &plength);
-        if ((pid == PID_SENTINEL) || !valid)
+        if (!valid || (pid == PID_SENTINEL))
         {
             break;
         }