[gstreamer-bugs] [Bug 399613] playbin: sink caps on vis-plugin not getting set (occasional segfault on AAC files)

[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=399613

  GStreamer | gst-plugins-base | Ver: 0.10.11





------- Comment #4 from Ed Catmur  2007-01-24 21:15 UTC -------
When playing oggs, gst_goom_sink_setcaps() gets called from:
#0  gst_goom_sink_setcaps (pad=0x83fc998, caps=0x84eaee0) at gstgoom.c:216
#1  0xb72ec857 in gst_pad_set_caps (pad=0x83fc998, caps=0x84eaee0)
    at gstpad.c:2328
#2  0xb72e12df in gst_proxy_pad_do_setcaps (pad=0x83982a0, caps=0x84eaee0)
    at gstghostpad.c:300
#3  0xb72ec857 in gst_pad_set_caps (pad=0x83982a0, caps=0x84eaee0)
    at gstpad.c:2328
#4  0xb72eddd8 in gst_pad_chain_unchecked (pad=0x83982a0, buffer=0xb1208e48)
    at gstpad.c:2387
#5  0xb72f2a4c in gst_pad_push (pad=0x8462248, buffer=0xb1208e48)
    at gstpad.c:3578
#6  0xb7357042 in gst_base_transform_chain (pad=0x8462188, buffer=0x8515908)
    at gstbasetransform.c:1548
...

Note that this is an optimised build; gstpad.c:2387 is actually in
gst_pad_configure_sink(). The problem is thus ll 3394-3399,

  /* we got a new datatype on the pad, see if it can handle it */
  if (G_UNLIKELY (caps_changed)) {
    GST_DEBUG_OBJECT (pad, "caps changed to %p %" GST_PTR_FORMAT, caps, caps);
    if (G_UNLIKELY (!gst_pad_configure_sink (pad, caps)))
      goto not_negotiated;
  }

caps_changed is not getting set when the first data comes through.

Ah... so the problem is that faad does not set caps on its buffers.  Is this
allowed?


-- 
Configure bugmail: http://bugzilla.gnome.org/userprefs.cgi?tab=email