[gstreamer-bugs] [Bug 319862] mpegstream/dvddemux partially ported to 0.9

[GStreamer (bugzilla.gnome.org)]

Do not reply to this via email (we are currently unable to handle email
responses and they get discarded).  You can add comments to this bug at
http://bugzilla.gnome.org/show_bug.cgi?id=319862
 GStreamer | gst-plugins-ugly | Ver: HEAD CVS





------- Additional Comments From Tim-Philipp Müller  2005-11-05 12:47 -------
Can't say this works too well for me:

With almost all .mpg files I get crashes like this:


(gst-launch-0.9:18655): GStreamer-CRITICAL **: gst_buffer_create_sub: assertion
`buffer->size >= offset + size' failed

Program received signal SIGSEGV, Segmentation fault.
[Switching to Thread -1217569872 (LWP 18658)]
gst_mpeg_demux_send_subbuffer (mpeg_demux=0x80da720, outstream=0x80cf448,
buffer=0x80e2850, timestamp=The value of variable 'timestamp' is distributed
across several
locations, and GDB cannot access its value.

) at gstmpegdemux.c:1051
1051      GST_BUFFER_TIMESTAMP (outbuf) = timestamp;

(in this case because outbuf is NULL because of the assertion failure of course).


The assertion isn't thrown all the time though. Sometimes it works to some
extent (with the same files). I've tried the alien file from
http://gstreamer.freedesktop.org/media/medium/alien.mpg and others.


I haven't looked at the code really, but I noticed that the return value of
gst_pad_push() is not always passed back to the caller. There were also
instances of GST_PAD_IS_USABLE() - it might be worth double checking that those
are used correctly in 0.9 - the usage has changed slightly (I _think_ that you
usually wouldn't use GST_PAD_IS_USABLE(), but just check the return value of
gst_pad_alloc_buffer() and bail out if that is not GST_FLOW_OK).

Cheers
 -Tim


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