Bug 340776 – [xvimagesink] totem crash sometime when opening a video or after playing it

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 340776 - [xvimagesink] totem crash sometime when opening a video or after playing it


Summary:	[xvimagesink] totem crash sometime when opening a video or after playing it


Status:	RESOLVED FIXED

Product:	GStreamer
Classification:	Platform
Component:	gst-plugins-base
Version:	0.10.6
Hardware:	Other Linux

Importance:	Normal critical
Target Milestone:	0.10.11
Assigned To:	GStreamer Maintainers
QA Contact:	GStreamer Maintainers

URL:
Whiteboard:

Depends on:
Blocks:

Reported:	2006-05-05 20:29 UTC by Sebastien Bacher
Modified:	2006-12-10 19:44 UTC

See Also:
GNOME target:	---
GNOME version:	2.13/2.14

Attachments
patch for the warnings (799 bytes, patch) 2006-05-13 16:37 UTC, Jan Schmidt	committed	Details \| Review

Description Sebastien Bacher 2006-05-05 20:29:45 UTC

I'm playing with totem after upgrading gst-plugins-good0.10 to 0.10.3 and it crashes often of some video when opening them or after playing them:

"(totem:7854): GLib-CRITICAL **: g_ascii_strcasecmp: assertion `s1 != NULL' failed

(totem:7854): GLib-CRITICAL **: g_ascii_strcasecmp: assertion `s1 != NULL' failed

(totem:7854): GLib-CRITICAL **: g_ascii_strcasecmp: assertion `s1 != NULL' failed
[Thread -1277043792 (LWP 7876) exited]

Program received signal SIGSEGV, Segmentation fault.
[Switching to Thread -1297921104 (LWP 7884)]
0xb6ae5990 in gst_xvimagesink_event_thread (xvimagesink=0x850cb68) at xvimagesink.c:934
934       while (XCheckWindowEvent (xvimagesink->xcontext->disp,
(gdb) thread apply all bt

+ Trace 68044

Thread 8 (Thread -1289528400 (LWP 7877))

#0 __kernel_vsyscall
#1 pthread_cond_signal
from /lib/tls/i686/cmov/libpthread.so.0
#2 gst_queue_loop
at gstqueue.c line 813
#3 gst_task_func
at gsttask.c line 193
#4 g_thread_pool_thread_proxy
at gthreadpool.c line 265
#5 g_thread_create_proxy
at gthread.c line 582
#6 start_thread
from /lib/tls/i686/cmov/libpthread.so.0
#7 clone
from /lib/tls/i686/cmov/libc.so.6

Thread 1 (Thread -1225476416 (LWP 7854))

#0 __kernel_vsyscall
#1 poll
from /lib/tls/i686/cmov/libc.so.6
#2 g_main_context_iterate
at gmain.c line 2849
#3 IA__g_main_loop_run
at gmain.c line 2751
#4 gst_bus_poll
at gstbus.c line 858
#5 bacon_video_widget_error_quark
#6 bacon_video_widget_error_quark
#7 bacon_video_widget_open_with_subtitle
#8 totem_action_set_mrl_with_warning
#9 totem_action_next
#10 IA__g_cclosure_marshal_VOID__VOID
at gmarshal.c line 77
#11 IA__g_closure_invoke
at gclosure.c line 490
#12 signal_emit_unlocked_R
at gsignal.c line 2438
#13 IA__g_signal_emit_valist
at gsignal.c line 2197
#14 IA__g_signal_emit
at gsignal.c line 2241
#15 bacon_video_widget_get_type
#16 g_idle_dispatch
at gmain.c line 3796
#17 IA__g_main_context_dispatch
at gmain.c line 1916
#18 g_main_context_iterate
at gmain.c line 2547
#19 IA__g_main_loop_run
at gmain.c line 2751
#20 IA__gtk_main
at gtkmain.c line 1026
#21 main

Comment 1 Sebastien Bacher 2006-05-05 20:35:39 UTC

Other backtrace:

Program received signal SIGSEGV, Segmentation fault.

+ Trace 68045

Thread NaN (LWP 8705)

#0 gst_xvimagesink_set_xwindow_id
at xvimagesink.c line 2148
#1 gst_x_overlay_set_xwindow_id
at xoverlay.c line 164
#2 bacon_video_widget_open_with_subtitle
#3 _gtk_marshal_BOOLEAN__BOXED
at gtkmarshalers.c line 83
#4 g_type_class_meta_marshal
at gclosure.c line 567
#5 IA__g_closure_invoke
at gclosure.c line 490
#6 signal_emit_unlocked_R
at gsignal.c line 2476
#7 IA__g_signal_emit_valist
at gsignal.c line 2207
#8 IA__g_signal_emit
at gsignal.c line 2241
#9 gtk_widget_event_internal
at gtkwidget.c line 3751
#10 IA__gtk_main_do_event
at gtkmain.c line 1409
#11 gdk_window_process_updates_internal
at gdkwindow.c line 2292
#12 IA__gdk_window_process_all_updates
at gdkwindow.c line 2345
#13 gtk_container_idle_sizer
at gtkcontainer.c line 1117
#14 g_idle_dispatch
at gmain.c line 3796
#15 IA__g_main_context_dispatch
at gmain.c line 1916
#16 g_main_context_iterate
at gmain.c line 2547
#17 IA__g_main_loop_run
at gmain.c line 2751
#18 gst_bus_poll
at gstbus.c line 858
#19 bacon_video_widget_error_quark
#20 bacon_video_widget_error_quark
#21 bacon_video_widget_open_with_subtitle
#22 totem_action_set_mrl_with_warning
#23 totem_action_next

Comment 2 Sebastien Bacher 2006-05-05 21:00:10 UTC

Happens with the previous version too, reassigning it

Comment 3 Tim-Philipp Müller 2006-05-05 21:08:35 UTC

Do you have to do anything in particular to cause this or does it just happen?

Also, could you do:

 $ G_DEBUG=fatal_warnings gdb --args totem /path/to/file

to find out where exactly that warning comes from?

Comment 4 Sebastien Bacher 2006-05-05 21:46:55 UTC

it looks like it happens when switching between videos or after playing one

The warnings are a different issue, should I open a different bug for them? Backtrace:

"GLib-CRITICAL **: g_ascii_strcasecmp: assertion `s1 != NULL' failed
aborting...

Program received signal SIGABRT, Aborted.

+ Trace 68049

Thread NaN (LWP 24806)

#0 __kernel_vsyscall
#1 raise
from /lib/tls/i686/cmov/libc.so.6
#2 abort
from /lib/tls/i686/cmov/libc.so.6
#3 IA__g_logv
#4 IA__g_log
at gmessages.c line 517
#5 IA__g_return_if_fail_warning
at gmessages.c line 532
#6 IA__g_ascii_strcasecmp
at gstrfuncs.c line 1761
#7 gst_value_deserialize_fraction
at gstvalue.c line 3381
#8 gst_value_transform_string_fraction
at gstvalue.c line 3403
#9 IA__g_value_transform
at gvalue.c line 355
#10 ev_sidebar_add_page
#11 bacon_video_widget_open_with_subtitle
#12 marshal_VOID__MINIOBJECT
at gstbus.c line 159
#13 IA__g_closure_invoke
at gclosure.c line 490
#14 signal_emit_unlocked_R
at gsignal.c line 2438
#15 IA__g_signal_emit_valist
at gsignal.c line 2197
#16 IA__g_signal_emit
at gsignal.c line 2241
#17 gst_bus_async_signal_func
at gstbus.c line 898
#18 gst_bus_source_dispatch
at gstbus.c line 625
#19 IA__g_main_context_dispatch
at gmain.c line 1916
#20 g_main_context_iterate
at gmain.c line 2547
#21 IA__g_main_loop_run
at gmain.c line 2751
#22 gst_bus_poll
#23 bacon_video_widget_error_quark
#24 bacon_video_widget_error_quark
#25 bacon_video_widget_open_with_subtitle
#26 totem_action_set_mrl_with_warning
#27 totem_action_next
#28 IA__g_cclosure_marshal_VOID__VOID
at gmarshal.c line 77
#29 IA__g_closure_invoke
at gclosure.c line 490
#30 signal_emit_unlocked_R
at gsignal.c line 2438
#31 IA__g_signal_emit_valist
at gsignal.c line 2197
#32 IA__g_signal_emit
at gsignal.c line 2241
#33 bacon_video_widget_get_type
#34 g_idle_dispatch
at gmain.c line 3796
#35 IA__g_main_context_dispatch
at gmain.c line 1916
#36 g_main_context_iterate
at gmain.c line 2547
#37 IA__g_main_loop_run
at gmain.c line 2751
#38 IA__gtk_main
at gtkmain.c line 1026
#39 main

Comment 5 Jan Schmidt 2006-05-13 16:37:10 UTC

Created attachment 65390 [details] [review]
patch for the warnings

This patch should fix the warnings about not being able to read a GstFraction from the pixel-aspect-ratio property of the video sink in comment #4

Comment 6 Jan Schmidt 2006-05-13 16:46:04 UTC

This looks like either a totem or an xvimagesink raciness bug to me (the segfault, not the g_value_transform warning)

Comment 7 Jan Schmidt 2006-05-26 09:21:28 UTC

Committed the patch to CVS. The race still needs finding.

Comment 8 Wim Taymans 2006-08-23 17:07:42 UTC

is this bug still present? can anyone trigger this and get a new backtrace?

Comment 9 André Klapper 2006-10-06 09:19:11 UTC

Anybody still facing this?

Comment 10 Sebastien Bacher 2006-10-07 14:07:14 UTC

I didn't get that issue for some time, marking as fixed