From 593615de46a98165ff34e74bb2559eeecec61fec Mon Sep 17 00:00:00 2001 From: Youness Alaoui Date: Fri, 3 Nov 2017 13:27:50 -0400 Subject: [PATCH] rtpg722pay: Add encoding-params to the src caps template The G722 payload only accepts G722 audio with channels=1, so it must specify the encoding-params=1 in its src caps, otherwise it causes issues with farstream which thinks it supports 2 channels G722 and when confronted with a remote that has G722/8000/2, it will negotiate it and error out with a not-negotiated when the caps don't intersect at runtime. https://bugzilla.gnome.org/show_bug.cgi?id=789878 --- gst/rtp/gstrtpg722pay.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/gst/rtp/gstrtpg722pay.c b/gst/rtp/gstrtpg722pay.c index c2fe88d..00b7f59 100644 --- a/gst/rtp/gstrtpg722pay.c +++ b/gst/rtp/gstrtpg722pay.c @@ -47,11 +47,13 @@ static GstStaticPadTemplate gst_rtp_g722_pay_src_template = "media = (string) \"audio\", " "encoding-name = (string) \"G722\", " "payload = (int) " GST_RTP_PAYLOAD_G722_STRING ", " + "encoding-params = (string) 1, " "clock-rate = (int) 8000; " "application/x-rtp, " "media = (string) \"audio\", " "encoding-name = (string) \"G722\", " "payload = (int) " GST_RTP_PAYLOAD_DYNAMIC_STRING ", " + "encoding-params = (string) 1, " "clock-rate = (int) 8000") ); -- 2.7.4