Commit cbd13883 authored by Stian Selnes's avatar Stian Selnes Committed by Sebastian Dröge

rtph263depay: Don't drop mode b packets with picture start code

Some buggy payloaders, e.g. rtph263pay, may use mode B for packets
that starts with a picture (or GOB) start code although it's not
allowed. Let's be nice and not drop these packets/frames.

https://bugzilla.gnome.org/show_bug.cgi?id=773516
parent 78ab8cbd
......@@ -310,9 +310,10 @@ gst_rtp_h263_depay_process (GstRTPBaseDepayload * depayload, GstRTPBuffer * rtp)
payload_len -= header_len;
if (!rtph263depay->start) {
/* do not skip this fragment if it is a Mode A with picture start code */
if (!F && payload_len > 4 && (GST_READ_UINT32_BE (payload) >> 10 == 0x20)) {
GST_DEBUG ("Mode A with PSC => frame start");
/* only mode A should be used when there is a picture start code, but
* buggy payloaders may send mode B/C in start of frame */
if (payload_len > 4 && (GST_READ_UINT32_BE (payload) >> 10 == 0x20)) {
GST_DEBUG ("Mode %c with PSC => frame start", "ABC"[F+P]);
rtph263depay->start = TRUE;
if ((! !(payload[4] & 0x02)) != I) {
GST_DEBUG ("Wrong Picture Coding Type Flag in rtp header");
......
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