Commit 999c63e4 authored by Adriano Pallavicino's avatar Adriano Pallavicino Committed by Martin Storsjö

rtp: Only choose static payload types if the sample rate and channels are right

If using a different sample rate or number of channels, use a dynamic
payload type instead, where the parameters are passed in the SDP.

G722 is a special case where the normal rules don't apply.
Signed-off-by: 's avatarMartin Storsjö <martin@martin.st>
parent d04c5293
...@@ -110,9 +110,17 @@ int ff_rtp_get_payload_type(AVFormatContext *fmt, AVCodecContext *codec) ...@@ -110,9 +110,17 @@ int ff_rtp_get_payload_type(AVFormatContext *fmt, AVCodecContext *codec)
!fmt->oformat->priv_class || !fmt->oformat->priv_class ||
!av_opt_flag_is_set(fmt->priv_data, "rtpflags", "rfc2190"))) !av_opt_flag_is_set(fmt->priv_data, "rtpflags", "rfc2190")))
continue; continue;
if (codec->codec_id == CODEC_ID_PCM_S16BE) /* G722 has 8000 as nominal rate even if the sample rate is 16000,
if (codec->channels != AVRtpPayloadTypes[i].audio_channels) * see section 4.5.2 in RFC 3551. */
continue; if (codec->codec_id == CODEC_ID_ADPCM_G722 &&
codec->sample_rate == 16000 && codec->channels == 1)
return AVRtpPayloadTypes[i].pt;
if (codec->codec_type == AVMEDIA_TYPE_AUDIO &&
((AVRtpPayloadTypes[i].clock_rate > 0 &&
codec->sample_rate != AVRtpPayloadTypes[i].clock_rate) ||
(AVRtpPayloadTypes[i].audio_channels > 0 &&
codec->channels != AVRtpPayloadTypes[i].audio_channels)))
continue;
return AVRtpPayloadTypes[i].pt; return AVRtpPayloadTypes[i].pt;
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment