[Bug 747965] rtppayload: payload type could be inconsistent in some payloader, which have pre-defined pt by RFC standard

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=747965

Thiago Sousa Santos <thiagossantos at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |thiagossantos at gmail.com

--- Comment #4 from Thiago Sousa Santos <thiagossantos at gmail.com> ---
(In reply to Hyunjun from comment #3)
> (In reply to Thiago Sousa Santos from comment #2)
> > 
> > ::: gst/rtp/gstrtpgsmpay.c
> > @@ +110,3 @@
> >  
> > +  if (!gst_structure_get_int (structure, "payload", &pt))
> > +    pt = GST_RTP_PAYLOAD_GSM;
> > 
> > This is the sink caps, can we have a "payload" here? Seems unusual.
> 
> Agree. 
> The existed logic to get payload type in setcaps function needs to be
> removed?
> (eg. g723pay, g729pay)

I think so, I wouldn't expect the sink caps to have the payload set.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.