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 689981 - avviddec: 0.9 port - opaque pointer sometimes NULL - causing crashes - make check fails
avviddec: 0.9 port - opaque pointer sometimes NULL - causing crashes - make c...
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-libav
git master
Other Linux
: Normal blocker
: 1.1.1
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2012-12-10 15:43 UTC by Tim-Philipp Müller
Modified: 2012-12-11 10:52 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description Tim-Philipp Müller 2012-12-10 15:43:07 UTC 
'make check' exposes this problem, but I also run into this every few times with a simple gst-launch-1.0 playbin pipeline (h264-in-matroska):

+ Trace 231278

Thread 4 (Thread 0x7f4d1297e700 (LWP 18084))

  • #0 nanosleep
    at ../sysdeps/unix/syscall-template.S line 82
  • #1 g_usleep
    at /build/glib2.0-7fLB_j/glib2.0-2.33.12+really2.32.4/./glib/gtimer.c line 259
  • #2 fault_spin
    at gst-launch.c line 149
  • #3 fault_handler_sigaction
    at gst-launch.c line 129
  • #4 <signal handler called>
  • #5 gst_ffmpegviddec_video_frame
    at gstavviddec.c line 1089
  • #6 gst_ffmpegviddec_frame
    at gstavviddec.c line 1195
  • #7 gst_ffmpegviddec_handle_frame
    at gstavviddec.c line 1312
  • #8 gst_video_decoder_decode_frame
    at gstvideodecoder.c line 2665
  • #9 gst_video_decoder_chain_forward
    at gstvideodecoder.c line 1697
  • #10 gst_video_decoder_chain
    at gstvideodecoder.c line 1957
  • #11 gst_pad_chain_data_unchecked
    at gstpad.c line 3654
  • #12 gst_pad_push_data
    at gstpad.c line 3871
  • #13 gst_pad_push
    at gstpad.c line 3974
  • #14 gst_base_transform_chain
    at gstbasetransform.c line 2203
  • #15 gst_pad_chain_data_unchecked
    at gstpad.c line 3654
  • #16 gst_pad_push_data
    at gstpad.c line 3871
  • #17 gst_pad_push
    at gstpad.c line 3974
  • #18 gst_base_parse_push_frame
    at gstbaseparse.c line 2142
  • #19 gst_base_parse_chain
    at gstbaseparse.c line 2614
  • #20 gst_pad_chain_data_unchecked
    at gstpad.c line 3654
  • #21 gst_pad_push_data
    at gstpad.c line 3871
  • #22 gst_pad_push
    at gstpad.c line 3974
  • #23 gst_single_queue_push_one
    at gstmultiqueue.c line 1057
  • #24 gst_multi_queue_loop
    at gstmultiqueue.c line 1303
  • #25 gst_task_func
    at gsttask.c line 316
  • #26 g_thread_pool_thread_proxy
    at /build/glib2.0-7fLB_j/glib2.0-2.33.12+really2.32.4/./glib/gthreadpool.c line 309
  • #27 g_thread_proxy
    at /build/glib2.0-7fLB_j/glib2.0-2.33.12+really2.32.4/./glib/gthread.c line 801
  • #28 start_thread
    at pthread_create.c line 304
  • #29 clone
    at ../sysdeps/unix/sysv/linux/x86_64/clone.S line 112
  • #30 ?? 






Comment 1


Sebastian Dröge (slomo)





          2012-12-11 10:52:10 UTC
        



commit 06de84c7477e9c917d44ff793aecd65e3a3226f5
Author: Sebastian Dröge <sebastian.droege@collabora.co.uk>
Date:   Tue Dec 11 10:34:42 2012 +0000

    avviddec: Fix memory leak if we already allocated an output buffer for the frame

commit 76b8b8c67f7c8fdf03d8d7d5c98b1e5fb83f50e0
Author: Wim Taymans <wim.taymans@collabora.co.uk>
Date:   Tue Dec 11 10:22:34 2012 +0100

    Revert "avviddec: Fix memory leaks and assertions in error cases if we can't allocate a frame"
    
    This reverts commit 47647e1caca117ff8a62caf8bf0b6c94eb6dd025.
    
    Breaks playback when direct rendering is disabled.
    The reason is that we set the opaque vaue to NULL and then try to use the NULL
    value when we decoded a frame.