Check whether we actually have a GroupId before trying to use it. (pr…

…oject-chip#36729) This avoids a crash on malformed messages (which will get dropped further down the pipeline, due to not being valid). Fixes project-chip#36711 Co-authored-by: Andrei Litvin <[email protected]>
yyzhong-g · Dec 11, 2024 · 37e53f6 · 37e53f6
1 parent e900b11
commit 37e53f6
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/src/messaging/ExchangeMgr.cpp b/src/messaging/ExchangeMgr.cpp
@@ -284,8 +284,15 @@ void ExchangeManager::OnMessageReceived(const PacketHeader & packetHeader, const
     }
     else
     {
-        ChipLogProgress(ExchangeManager, "Received Groupcast Message with GroupId 0x%04X (%d)",
-                        packetHeader.GetDestinationGroupId().Value(), packetHeader.GetDestinationGroupId().Value());
+        if (packetHeader.GetDestinationGroupId().HasValue())
+        {
+            ChipLogProgress(ExchangeManager, "Received Groupcast Message with GroupId 0x%04X (%d)",
+                            packetHeader.GetDestinationGroupId().Value(), packetHeader.GetDestinationGroupId().Value());
+        }
+        else
+        {
+            ChipLogProgress(ExchangeManager, "Received Groupcast Message without GroupId");
+        }
     }
 
     // Do not handle messages that don't match an existing exchange on an