Bug 583867 – gdpdepay + identity cause failed assertions

After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.

Bug 583867 - gdpdepay + identity cause failed assertions


Summary:	gdpdepay + identity cause failed assertions


Status:	RESOLVED FIXED

Product:	GStreamer
Classification:	Platform
Component:	gst-plugins-base
Version:	git master
Hardware:	Other Linux

Importance:	Normal normal
Target Milestone:	0.10.24
Assigned To:	GStreamer Maintainers
QA Contact:	GStreamer Maintainers

URL:
Whiteboard:

Depends on:
Blocks:

Reported:	2009-05-26 09:09 UTC by Jan Urbański
Modified:	2009-05-27 10:11 UTC

See Also:
GNOME target:	---
GNOME version:	---

Description Jan Urbański 2009-05-26 09:09:58 UTC

With the following pipelines (and latest git):

  gst-launch audiotestsrc ! vorbisenc ! oggmux ! gdppay ! identity ! fakesink
  gst-launch videotestsrc ! theoraenc ! oggmux ! gdppay ! identity ! fakesink

I get lots of errors like that:

  gst_caps_is_subset: assertion `subset != NULL' failed
  gst_caps_is_fixed: assertion `GST_IS_CAPS (caps)' failed
  identity0: could not get in_size

After running with --gst-fatal-warnings the following backtrace can be obtained from the core:

(...)

+ Trace 215702

#6 gst_base_transform_handle_buffer
at gstbasetransform.c line 1268
#7 gst_base_transform_chain
at gstbasetransform.c line 2018
#8 gst_pad_chain_data_unchecked
at gstpad.c line 4047
#9 gst_pad_push_data
at gstpad.c line 4277
#10 gst_gdp_pay_reset_streamheader
at gstgdppay.c line 491


However, none of these pipelines gives any errors:

  gst-launch audiotestsrc ! vorbisenc ! oggmux ! gdppay ! identity
  gst-launch audiotestsrc ! vorbisenc ! oggmux ! gdppay ! fakesink
  gst-launch audiotestsrc ! vorbisenc ! oggmux ! gdppay ! gdpdepay ! identity ! fakesink

Comment 1 Wim Taymans 2009-05-27 10:11:10 UTC

commit 4fad9390539a4d0544dd953481182b6b7dbefe77
Author: Wim Taymans <wim.taymans@collabora.co.uk>
Date:   Wed May 27 12:10:05 2009 +0200

    gdppay: set caps on outgoing buffers
    
    Set caps on outgoing buffers because NULL caps confuse basetransform.
    Fixes #583867