sdp: Make opus declaration conform to the spec
authorTimothy B. Terriberry <tterribe@xiph.org>
Thu, 25 Sep 2014 00:43:22 +0000 (17:43 -0700)
committerMartin Storsjö <martin@martin.st>
Mon, 29 Sep 2014 10:48:42 +0000 (13:48 +0300)
Signed-off-by: Martin Storsjö <martin@martin.st>
libavformat/sdp.c

index 3c39ac7..eccd676 100644 (file)
@@ -583,8 +583,18 @@ static char *sdp_write_media_attributes(char *buff, int size, AVCodecContext *c,
                                      payload_type, c->sample_rate);
             break;
         case AV_CODEC_ID_OPUS:
-            av_strlcatf(buff, size, "a=rtpmap:%d opus/48000\r\n",
+            /* The opus RTP draft says that all opus streams MUST be declared
+               as stereo, to avoid negotiation failures. The actual number of
+               channels can change on a packet-by-packet basis. The number of
+               channels a receiver prefers to receive or a sender plans to send
+               can be declared via fmtp parameters (both default to mono), but
+               receivers MUST be able to receive and process stereo packets. */
+            av_strlcatf(buff, size, "a=rtpmap:%d opus/48000/2\r\n",
                                      payload_type);
+            if (c->channels == 2) {
+                av_strlcatf(buff, size, "a=fmtp:%d sprop-stereo:1\r\n",
+                                         payload_type);
+            }
             break;
         default:
             /* Nothing special to do here... */