From 6da8eeb946b8f3dfb7bc0933ba726ad16929bd06 Mon Sep 17 00:00:00 2001 From: "punyabrata@webrtc.org" Date: Tue, 24 Jan 2012 00:48:36 +0000 Subject: [PATCH] Removing an assert for a case that can occur when corrupt packets are injected into voice engine. Review URL: https://webrtc-codereview.appspot.com/373004 git-svn-id: http://webrtc.googlecode.com/svn/trunk@1518 4adac7df-926f-26a2-2b94-8c16560cd09d --- src/modules/rtp_rtcp/source/receiver_fec.cc | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/modules/rtp_rtcp/source/receiver_fec.cc b/src/modules/rtp_rtcp/source/receiver_fec.cc index 34e127f88f..2a8820836d 100644 --- a/src/modules/rtp_rtcp/source/receiver_fec.cc +++ b/src/modules/rtp_rtcp/source/receiver_fec.cc @@ -124,10 +124,11 @@ WebRtc_Word32 ReceiverFEC::AddReceivedFECPacket( timestampOffset += incomingRtpPacket[rtpHeader->header.headerLength+2]; timestampOffset = timestampOffset >> 2; if(timestampOffset != 0) { - // sanity timestampOffset must be 0 - assert(false); + // timestampOffset should be 0, however, this is a valid error case in + // the event of garbage payload. return -1; } + blockLength = (0x03 & incomingRtpPacket[rtpHeader->header.headerLength + 2]) << 8; blockLength += (incomingRtpPacket[rtpHeader->header.headerLength + 3]);