zqz/platform-external-webrtc
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

sdp: reject duplicate codecs with the same id but different name or clockrate

Browse Source 
since something like
  rtpmap:96 VP8/90000
  rtpmap:96 VP9/90000
or
  rtpmap:97 ISAC/32000
  rtpmap:97 ISAC/16000
is wrong. Note that fmtp or rtcp-fb are not taken into account.
Also note that sending invalid static payload types now throws an error.

Drive-by: replace "RtpMap" with "Rtpmap" for consistency.

BUG=None

Change-Id: I2574b82a6f1a0afe3edc866e514a5dbca0798e8c
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/263641
Reviewed-by: Harald Alvestrand <hta@webrtc.org>
Commit-Queue: Philipp Hancke <philipp.hancke@googlemail.com>
Cr-Commit-Position: refs/heads/main@{#37028}
This commit is contained in:
Philipp Hancke
2022-05-30 08:45:41 +02:00
committed by WebRTC LUCI CQ
parent fc2c24ef44
commit ad6807805d
3 changed files with 59 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

30
pc/webrtc_sdp.cc
View File

@ -280,7 +280,7 @@ static void BuildRtpContentAttributes(const MediaContentDescription* media_desc,
const cricket::MediaType media_type,
int msid_signaling,
std::string* message);
static void BuildRtpMap(const MediaContentDescription* media_desc,
static void BuildRtpmap(const MediaContentDescription* media_desc,
const cricket::MediaType media_type,
std::string* message);
static void BuildCandidate(const std::vector<Candidate>& candidates,
@ -1682,7 +1682,7 @@ void BuildRtpContentAttributes(const MediaContentDescription* media_desc,
// RFC 4566
// a=rtpmap:<payload type> <encoding name>/<clock rate>
// [/<encodingparameters>]
BuildRtpMap(media_desc, media_type, message);
BuildRtpmap(media_desc, media_type, message);
for (const StreamParams& track : media_desc->streams()) {
// Build the ssrc-group lines.
@ -1882,7 +1882,7 @@ bool GetParameter(const std::string& name,
return true;
}
void BuildRtpMap(const MediaContentDescription* media_desc,
void BuildRtpmap(const MediaContentDescription* media_desc,
const cricket::MediaType media_type,
std::string* message) {
RTC_DCHECK(message != NULL);
@ -2852,7 +2852,7 @@ T GetCodecWithPayloadType(const std::vector<T>& codecs, int payload_type) {
return ret_val;
}
// Updates or creates a new codec entry in the audio description.
// Updates or creates a new codec entry in the media description.
template <class T, class U>
void AddOrReplaceCodec(MediaContentDescription* content_desc, const U& codec) {
T* desc = static_cast<T*>(content_desc);
@ -3596,8 +3596,19 @@ bool ParseRtpmapAttribute(absl::string_view line,
if (!GetValueFromString(line, codec_params[1], &clock_rate, error)) {
return false;
}
if (media_type == cricket::MEDIA_TYPE_VIDEO) {
VideoContentDescription* video_desc = media_desc->as_video();
for (const cricket::VideoCodec& existing_codec : video_desc->codecs()) {
if (payload_type == existing_codec.id &&
(encoding_name != existing_codec.name ||
clock_rate != existing_codec.clockrate)) {
return ParseFailed(
line,
"Duplicate payload type with conflicting codec name or clock rate.",
error);
}
}
UpdateCodec(payload_type, encoding_name, video_desc);
} else if (media_type == cricket::MEDIA_TYPE_AUDIO) {
// RFC 4566
@ -3616,6 +3627,17 @@ bool ParseRtpmapAttribute(absl::string_view line,
}
AudioContentDescription* audio_desc = media_desc->as_audio();
for (const cricket::AudioCodec& existing_codec : audio_desc->codecs()) {
if (payload_type == existing_codec.id &&
(encoding_name != existing_codec.name ||
clock_rate != existing_codec.clockrate ||
channels != existing_codec.channels)) {
return ParseFailed(line,
"Duplicate payload type with conflicting codec "
"name, clock rate or number of channels.",
error);
}
}
UpdateCodec(payload_type, encoding_name, clock_rate, 0, channels,
audio_desc);
}