[Bug 753323] mpegts: add support for JPEG 2000 to mpegtsmux and tsdemux

[GStreamer (GNOME Bugzilla)]

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

--- Comment #147 from Aaron Boxer <boxerab at gmail.com> ---
So, I found that I could get PES packets of size greater than 0xffff
if I commented out the following lines in tsmux_stream_initialize_pes_packet
and force cur_pes_payload_size equal to zero.



  if (stream->is_video_stream) {
    //guint8 hdr_len;

    //hdr_len = tsmux_stream_pes_header_length (stream);

    /* Unbounded for video streams if pes packet length is over 16 bit */
    //if ((stream->cur_pes_payload_size + hdr_len - 6) > G_MAXUINT16)
      stream->cur_pes_payload_size = 0;
  }


This code should have been triggered anyways, without my changes, so it looks
to me like a bug with the code that calculates cur_pes_payload_size ?.


The pipeline is

gst-launch-1.0 -v videotestsrc  ! video/x-raw,width=1000,height=1000 !
openjpegenc ! jpeg2000parse ! mpegtsmux ! tsdemux ! jpeg2000parse ! openjpegdec
! videoconvert ! ximagesink



with my patch

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