h264depay: handle properly STAPs
authorMarco Ballesio <marco.ballesio@nokia.com>
Wed, 7 Apr 2010 13:06:54 +0000 (16:06 +0300)
committerStefan Kost <ensonic@users.sf.net>
Wed, 7 Apr 2010 13:17:06 +0000 (16:17 +0300)
in rtph264depay.c, lines 577-576, NALU-type 24 (Single-Time Aggregation
Packet) is handled in fall-through as NALU-type 26 (unhandled).

This leads high quality h264 streams such as:

rtsp://stream.yle.mobi/yle/areena/MEDIA_E0342657_p3.mp4

to fail with "NAL unit type 24 not supported yet" (but it's actually
supported), and thus to close any stream which contains STAPs.

The proposed one-liner patch fixes the issue.
Fixes #615051.

gst/rtp/gstrtph264depay.c

index be88f29..44f8758 100644 (file)
@@ -574,6 +574,7 @@ gst_rtp_h264_depay_process (GstBaseRTPDepayload * depayload, GstBuffer * buf)
           gst_buffer_set_caps (outbuf, GST_PAD_CAPS (depayload->srcpad));
           return outbuf;
         }
+        break;
       }
       case 26:
         /* MTAP16    Multi-time aggregation packet      5.7.2 */