From 951e2898535671249fa179ff9834dba7d5a39fe4 Mon Sep 17 00:00:00 2001 From: Bjorn A Mellem Date: Mon, 9 Dec 2019 09:37:42 -0800 Subject: [PATCH] Add VideoTimingExtension to kFecOrPaddingExtensionSizes. As of https://webrtc-review.googlesource.com/c/src/+/158899, FEC may be used on packets with VideoTimingExtension. This may result in creation of FEC packets that exceed the maximum configured RTP packet size. This problem occurs most frequently with datagram transports that define a smaller maximum packet size. Bug: webrtc:9719 Change-Id: I842216a6696a695f0a3f01a221e538605fc5b9bd Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/161557 Reviewed-by: Danil Chapovalov Commit-Queue: Bjorn Mellem Cr-Commit-Position: refs/heads/master@{#30045} --- modules/rtp_rtcp/source/rtp_sender.cc | 1 + 1 file changed, 1 insertion(+) diff --git a/modules/rtp_rtcp/source/rtp_sender.cc b/modules/rtp_rtcp/source/rtp_sender.cc index d7622f8835..6010d033b7 100644 --- a/modules/rtp_rtcp/source/rtp_sender.cc +++ b/modules/rtp_rtcp/source/rtp_sender.cc @@ -63,6 +63,7 @@ constexpr RtpExtensionSize kFecOrPaddingExtensionSizes[] = { CreateExtensionSize(), CreateExtensionSize(), CreateMaxExtensionSize(), + CreateExtensionSize(), }; // Size info for header extensions that might be used in video packets.