[Bug 723196] Incorrect calculation of rtp header length

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=723196
  GStreamer | gst-plugins-good | 1.x

Wim Taymans <wim.taymans> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEEDINFO
                 CC|                            |wim.taymans at gmail.com

--- Comment #1 from Wim Taymans <wim.taymans at gmail.com> 2014-02-04 13:22:42 UTC ---
> It looks like the rtppcmudepay module incorrectly calculated the rtp header
> size.  It assumed it was always 12 bytes, yet when there are contributing
> source identifiers this is incorrect.

Where do you see it do that? The code looks like it all handles this correctly,
look at:

gst_rtp_buffer_get_header_len() and
gst_rtp_buffer_map()

> I confirmed that this was the problem because I had audio artifacts before I
> reformatted the RTP packet and stripped out the contributing source identifiers
> and no artifacts afterwards.

I think you have a different problem then. Can you provide a dump of the RTP
packet?

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.