Skip to content

Commit

Permalink
Remove redundant duplicate message logging. (#9967)
Browse files Browse the repository at this point in the history
We already log the duplicate at the point when we actually drop it.
  • Loading branch information
bzbarsky-apple authored and pull[bot] committed Oct 18, 2021
1 parent a07b5f7 commit 1568102
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions src/transport/SessionManager.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -352,7 +352,6 @@ void SessionManager::MessageDispatch(const PacketHeader & packetHeader, const Tr
CHIP_ERROR err = session->GetPeerMessageCounter().VerifyOrTrustFirst(packetHeader.GetMessageCounter());
if (err == CHIP_ERROR_DUPLICATE_MESSAGE_RECEIVED)
{
ChipLogDetail(Inet, "Received a duplicate message with MessageCounter: %" PRIu32, packetHeader.GetMessageCounter());
isDuplicate = SessionManagerDelegate::DuplicateMessage::Yes;
err = CHIP_NO_ERROR;
}
Expand Down Expand Up @@ -433,7 +432,6 @@ void SessionManager::SecureMessageDispatch(const PacketHeader & packetHeader, co
err = state->GetSessionMessageCounter().GetPeerMessageCounter().Verify(packetHeader.GetMessageCounter());
if (err == CHIP_ERROR_DUPLICATE_MESSAGE_RECEIVED)
{
ChipLogDetail(Inet, "Received a duplicate message with MessageCounter: %" PRIu32, packetHeader.GetMessageCounter());
isDuplicate = SessionManagerDelegate::DuplicateMessage::Yes;
err = CHIP_NO_ERROR;
}
Expand Down

0 comments on commit 1568102

Please sign in to comment.