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 694717 - gst_memory_share: assertion `!GST_MEMORY_FLAG_IS_SET (mem, GST_MEMORY_FLAG_NO_SHARE)' failed in flacparse unit test
gst_memory_share: assertion `!GST_MEMORY_FLAG_IS_SET (mem, GST_MEMORY_FLAG_NO...
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gstreamer (core)
git master
Other Linux
: Normal blocker
: 1.1.1
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2013-02-26 00:31 UTC by Tim-Philipp Müller
Modified: 2013-02-26 08:10 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Tim-Philipp Müller 2013-02-26 00:31:37 UTC
Recent core changes make the flacparse unit test in -good abort like this:

Running suite(s): flacparse

(lt-flacparse:1683): GStreamer-CRITICAL **: gst_memory_share: assertion `!GST_MEMORY_FLAG_IS_SET (mem, GST_MEMORY_FLAG_NO_SHARE)' failed

Program received signal SIGTRAP, Trace/breakpoint trap.
g_logv (log_domain=0x2aaaaaf90f98 "GStreamer", log_level=G_LOG_LEVEL_CRITICAL, format=<optimized out>, args=args@entry=0x7fffffffb238) at gmessages.c:981
981		  g_private_set (&g_log_depth, GUINT_TO_POINTER (depth));
(gdb) bt
  • #0 g_logv
    at gmessages.c line 981
  • #1 g_log
    at gmessages.c line 1010
  • #2 g_return_if_fail_warning
  • #3 gst_memory_share
    at gstmemory.c line 360
  • #4 gst_buffer_copy_into
    at gstbuffer.c line 430
  • #5 gst_buffer_copy_region
    at gstbuffer.c line 1742
  • #6 gst_flac_parse_parse_frame
    at gstflacparse.c line 1497
  • #7 gst_flac_parse_handle_frame
    at gstflacparse.c line 861
  • #8 gst_base_parse_handle_buffer
    at gstbaseparse.c line 1881
  • #9 gst_base_parse_chain
    at gstbaseparse.c line 2740
  • #10 gst_pad_chain_data_unchecked
    at gstpad.c line 3655
  • #11 gst_pad_push_data
    at gstpad.c line 3872
  • #12 gst_pad_push
    at gstpad.c line 3975
  • #13 gst_parser_test_run
    at elements/parser.c line 242
  • #14 gst_parser_test_normal
    at elements/parser.c line 318
  • #15 test_parse_flac_normal
    at elements/flacparse.c line 129
  • #16 tcase_run_tfun_nofork
    at check_run.c line 314
  • #17 srunner_iterate_tcase_tfuns
    at check_run.c line 181
  • #18 srunner_run_tcase
    at check_run.c line 302
  • #19 srunner_iterate_suites
    at check_run.c line 150
  • #20 srunner_run_all
    at check_run.c line 561
  • #21 main
    at elements/flacparse.c line 295



Could of course also be a baseparse or flacparse bug that was only exposed by those changes. I have not investigated.
Comment 1 Sebastian Dröge (slomo) 2013-02-26 08:10:13 UTC
commit 2ca00469347487c31946fba36e75ca8d2be010ce
Author: Sebastian Dröge <sebastian.droege@collabora.co.uk>
Date:   Tue Feb 26 09:08:49 2013 +0100

    buffer: Fix inverted logic for deciding if memory should be shared or copied
    
    https://bugzilla.gnome.org/show_bug.cgi?id=694717