From fc715f584fbf53f0e24d75806c3ae9c3cda76d94 Mon Sep 17 00:00:00 2001 From: stefan Date: Fri, 3 Jun 2016 01:45:36 -0700 Subject: [PATCH] DCHECK that the red payload type doesn't have invalid values when FEC is enabled. Review-Url: https://codereview.webrtc.org/2036083002 Cr-Commit-Position: refs/heads/master@{#13028} --- webrtc/modules/rtp_rtcp/source/rtp_sender_video.cc | 1 + 1 file changed, 1 insertion(+) diff --git a/webrtc/modules/rtp_rtcp/source/rtp_sender_video.cc b/webrtc/modules/rtp_rtcp/source/rtp_sender_video.cc index e10b5b2eda..3dab6a4bcb 100644 --- a/webrtc/modules/rtp_rtcp/source/rtp_sender_video.cc +++ b/webrtc/modules/rtp_rtcp/source/rtp_sender_video.cc @@ -169,6 +169,7 @@ void RTPSenderVideo::SendVideoPacketAsRed(uint8_t* data_buffer, void RTPSenderVideo::SetGenericFECStatus(const bool enable, const uint8_t payloadTypeRED, const uint8_t payloadTypeFEC) { + RTC_DCHECK(!enable || payloadTypeRED > 0); rtc::CritScope cs(&crit_); fec_enabled_ = enable; red_payload_type_ = payloadTypeRED;