Bug 329181 – totem crash when using goom effect

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 329181 - totem crash when using goom effect


Summary:	totem crash when using goom effect


Status:	RESOLVED FIXED

Product:	GStreamer
Classification:	Platform
Component:	gst-plugins-good
Version:	0.10.1
Hardware:	Other Linux

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

URL:
Whiteboard:

Depends on:
Blocks:

Reported:	2006-01-30 09:57 UTC by Sebastien Bacher
Modified:	2006-02-06 15:05 UTC

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

Description Sebastien Bacher 2006-01-30 09:57:51 UTC

With totem 1.3.90 and gst-plugins-good0.10 0.10.2/goom effect used:

Program received signal SIGSEGV, Segmentation fault.

+ Trace 65737

Thread NaN (LWP 11352)

#0 goom_update
at goom_core.c line 318
#1 gst_goom_chain
at gstgoom.c line 385
#2 gst_pad_chain
from /usr/lib/libgstreamer-0.10.so.0
#3 gst_proxy_pad_get_type
from /usr/lib/libgstreamer-0.10.so.0
#4 gst_pad_chain
from /usr/lib/libgstreamer-0.10.so.0
#5 gst_pad_push
from /usr/lib/libgstreamer-0.10.so.0
#6 gst_base_transform_set_passthrough
from /usr/lib/libgstbase-0.10.so.0
#7 gst_pad_chain
from /usr/lib/libgstreamer-0.10.so.0
#8 gst_pad_push
from /usr/lib/libgstreamer-0.10.so.0
#9 gst_tee_get_type
from /usr/lib/gstreamer-0.10/libgstcoreelements.so
#10 gst_iterator_fold
from /usr/lib/libgstreamer-0.10.so.0
#11 gst_tee_get_type
from /usr/lib/gstreamer-0.10/libgstcoreelements.so
#12 gst_pad_chain
from /usr/lib/libgstreamer-0.10.so.0
#13 gst_proxy_pad_get_type
from /usr/lib/libgstreamer-0.10.so.0
#14 gst_pad_chain
from /usr/lib/libgstreamer-0.10.so.0
#15 gst_pad_push
from /usr/lib/libgstreamer-0.10.so.0
#16 gst_queue_get_type
from /usr/lib/gstreamer-0.10/libgstcoreelements.so
#17 gst_task_get_type
from /usr/lib/libgstreamer-0.10.so.0
#18 g_thread_pool_free
from /usr/lib/libglib-2.0.so.0
#19 g_static_private_free
from /usr/lib/libglib-2.0.so.0
#20 start_thread
from /lib/tls/i686/cmov/libpthread.so.0
#21 clone
from /lib/tls/i686/cmov/libc.so.6

Comment 1 Baptiste Mille-Mathias 2006-01-30 18:28:05 UTC

Confirm here,

it appeared when I changed the size of the goom effect while playing a track with goom.

Comment 2 Gad Kadosh 2006-02-04 21:24:12 UTC

Same problem here

Comment 3 Jan Schmidt 2006-02-05 20:47:12 UTC

Fixed in CVS of gst-plugins-good:

        * gst/goom/filters.c: (zoomFilterNew), (calculatePXandPY),
        (setPixelRGB), (setPixelRGB_), (getPixelRGB), (getPixelRGB_),
        (zoomFilterSetResolution), (zoomFilterDestroy),
        (zoomFilterFastRGB), (pointFilter):
        * gst/goom/filters.h:
        * gst/goom/goom_core.c: (goom_init), (goom_set_resolution),
        (goom_update), (goom_close):
        * gst/goom/goom_core.h:
        * gst/goom/goom_tools.h:
        * gst/goom/graphic.c:
        * gst/goom/gstgoom.c: (gst_goom_class_init), (gst_goom_init),
        (gst_goom_dispose), (gst_goom_src_setcaps), (gst_goom_chain):
        * gst/goom/gstgoom.h:
        * gst/goom/lines.c: (goom_lines):
        * gst/goom/lines.h:
          Make goom reentrant by moving all important static variables
          into instance structures.
          (Fixes #329181)