commit 7d1aff6eed05fdb9daabb06259c55d2049e41742
author Philipp Hancke <phancke@microsoft.com> Mon Sep 25 12:42:51 2023
committer WebRTC LUCI CQ <webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com> Mon Sep 25 14:54:22 2023
tree c87858e04f2e80725436ad00465d8a62a743eadc
parent 6bf2d31e71320c2db06495ea35b82f613426dd50

Unify RTP payload type validity checking

making the UsedId generator the source of truth.
BUG=webrtc:12197

Change-Id: I4318a1366f8b2e20ea5ae264232437a9006c5103
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/321120
Commit-Queue: Philipp Hancke <phancke@microsoft.com>
Reviewed-by: Henrik Boström <hbos@webrtc.org>
Reviewed-by: Harald Alvestrand <hta@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#40802}

pc/sdp_offer_answer.cc

diff --git a/pc/sdp_offer_answer.cc b/pc/sdp_offer_answer.cc
index fe7093d..1ed5c80 100644
--- a/pc/sdp_offer_answer.cc
+++ b/pc/sdp_offer_answer.cc
@@ -53,6 +53,7 @@
 #include "pc/rtp_sender_proxy.h"
 #include "pc/simulcast_description.h"
 #include "pc/usage_pattern.h"
+#include "pc/used_ids.h"
 #include "pc/webrtc_session_description_factory.h"
 #include "rtc_base/helpers.h"
 #include "rtc_base/logging.h"
@@ -120,12 +121,6 @@
 static const char kDefaultAudioSenderId[] = "defaulta0";
 static const char kDefaultVideoSenderId[] = "defaultv0";
 
-// NOTE: Duplicated from pc/used_ids.h
-static const int kLastDynamicPayloadTypeLowerRange = 63;
-
-static const int kFirstDynamicPayloadTypeUpperRange = 96;
-static const int kLastDynamicPayloadTypeUpperRange = 127;
-
 void NoteAddIceCandidateResult(int result) {
   RTC_HISTOGRAM_ENUMERATION("WebRTC.PeerConnection.AddIceCandidate", result,
                             kAddIceCandidateMax);
@@ -574,10 +569,8 @@
     if (type == cricket::MEDIA_TYPE_AUDIO) {
       RTC_DCHECK(media_description->as_audio());
       for (const auto& codec : media_description->as_audio()->codecs()) {
-        if (codec.id < 0 || codec.id > kLastDynamicPayloadTypeUpperRange ||
-            (media_description->rtcp_mux() &&
-             (codec.id > kLastDynamicPayloadTypeLowerRange &&
-              codec.id < kFirstDynamicPayloadTypeUpperRange))) {
+        if (!cricket::UsedPayloadTypes::IsIdValid(
+                codec, media_description->rtcp_mux())) {
           LOG_AND_RETURN_ERROR(
               RTCErrorType::INVALID_PARAMETER,
               "The media section with MID='" + content.mid() +
@@ -589,10 +582,8 @@
     } else if (type == cricket::MEDIA_TYPE_VIDEO) {
       RTC_DCHECK(media_description->as_video());
       for (const auto& codec : media_description->as_video()->codecs()) {
-        if (codec.id < 0 || codec.id > kLastDynamicPayloadTypeUpperRange ||
-            (media_description->rtcp_mux() &&
-             (codec.id > kLastDynamicPayloadTypeLowerRange &&
-              codec.id < kFirstDynamicPayloadTypeUpperRange))) {
+        if (!cricket::UsedPayloadTypes::IsIdValid(
+                codec, media_description->rtcp_mux())) {
           LOG_AND_RETURN_ERROR(
               RTCErrorType::INVALID_PARAMETER,
               "The media section with MID='" + content.mid() +