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 668664 - [0.11] taglist: g_date_time_ref warning
[0.11] taglist: g_date_time_ref warning
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
0.11.x
Other Linux
: Normal normal
: 0.11.x
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2012-01-25 14:09 UTC by Philippe Normand
Modified: 2012-01-26 18:29 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description Philippe Normand 2012-01-25 14:09:29 UTC 
0.10 doesn't seem to be affected by this issue, I need to double-check though.

G_DEBUG=fatal_criticals gdb -args gst-launch-0.11 playbin uri=http://dl.dropbox.com/u/280087/counting.mp4

GLib-CRITICAL **: g_date_time_ref: assertion `datetime->ref_count > 0' failed


(gdb) bt

+ Trace 229515

  • #0 g_logv
    at gmessages.c line 765
  • #1 g_log
    at gmessages.c line 792
  • #2 g_return_if_fail_warning
    at gmessages.c line 801
  • #3 g_date_time_ref
    at gdatetime.c line 453
  • #4 _g_type_boxed_copy
    at gtype.c line 4238
  • #5 boxed_proxy_collect_value
    at gboxed.c line 234
  • #6 boxed_proxy_collect_value
    at gboxed.c line 219
  • #7 gst_tag_list_add_valist
    at gsttaglist.c line 1183
  • #8 gst_tag_list_add
    at gsttaglist.c line 1123
  • #9 qtdemux_parse_tree
    at qtdemux.c line 8874
  • #10 gst_qtdemux_loop_state_header
    at qtdemux.c line 2740
  • #11 gst_qtdemux_loop
    at qtdemux.c line 3807
  • #12 gst_task_func
    at gsttask.c line 314
  • #13 default_func
    at gsttaskpool.c line 70
  • #14 g_thread_pool_thread_proxy
    at gthreadpool.c line 317
  • #15 g_thread_proxy
    at gthread.c line 801
  • #16 start_thread
    at pthread_create.c line 304 






Comment 1


Philippe Normand





          2012-01-25 14:46:07 UTC
        



Yep, works fine with 0.10 git master.






Comment 2


Vincent Penquerc'h





          2012-01-25 18:25:14 UTC
        



Reproduced. Some mixup between GDatetime and GstDatetime.






Comment 3


Vincent Penquerc'h





          2012-01-26 17:57:03 UTC
        



GST_TAG_DATE_TIME now points to G_TYPE_DATE_TIME, instead of GST_TYPE_DATE_TIME.
A similar patch was done to avidemux, though I don't have a file to test that one with.
Sorry, I forgot to include a link to this bug in the commit :/


commit 53261261120b4c008de61691c70e94354b28004a
Author: Vincent Penquerc'h <vincent.penquerch@collabora.co.uk>
Date:   Thu Jan 26 17:47:29 2012 +0000

    qtdemux: fix GstDateTime/GDateTime mixup






Comment 4


Vincent Penquerc'h





          2012-01-26 18:29:21 UTC
        



It turns out that the change in core was accidental, so I reverted it, and those two patches as well.