Bug 662463 – gnome-system-monitor crashes when nicotine+ is running

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 662463 - gnome-system-monitor crashes when nicotine+ is running


Summary:	gnome-system-monitor crashes when nicotine+ is running


Status:	RESOLVED FIXED

Product:	system-monitor
Classification:	Core
Component:	general
Version:	3.2.x
Hardware:	Other Linux

Importance:	High critical
Target Milestone:	---
Assigned To:	Robert Roth
QA Contact:	System-monitor maintainers

URL:
Whiteboard:

Depends on:
Blocks:

Reported:	2011-10-22 15:22 UTC by Guillaume Mureu
Modified:	2012-01-08 22:26 UTC

See Also:	https://launchpad.net/bugs/700900
GNOME target:	---
GNOME version:	3.1/3.2

Attachments
Proposed patch (1.04 KB, patch) 2012-01-03 22:00 UTC, Robert Roth	none	Details \| Review
Proposed patch v2 (1.08 KB, patch) 2012-01-03 22:18 UTC, Robert Roth	none	Details \| Review

Description Guillaume Mureu 2011-10-22 15:22:01 UTC

1) Have nicotine+ running in the background.
2) Launch the system monitor either from the CLI or the panel.
3) gnome-system-monitor's window flashes and it crashes.

--

$ gdb gnome-system-monitor 
GNU gdb (Ubuntu/Linaro 7.3-0ubuntu2) 7.3-2011.08
This GDB was configured as "x86_64-linux-gnu".
Reading symbols from /usr/bin/gnome-system-monitor...(no debugging symbols found)...done.
(gdb) run
Starting program: /usr/bin/gnome-system-monitor 
[Thread debugging using libthread_db enabled]
[New Thread 0x7fffeb457700 (LWP 4532)]
[New Thread 0x7fffeac56700 (LWP 4533)]
[New Thread 0x7fffe933b700 (LWP 4534)]

** (gnome-system-monitor:4529): WARNING **: SELinux was found but is not enabled.


** (gnome-system-monitor:4529): WARNING **: Failed to load icon for nicotine(4048) : Error opening file: No such file or directory

** (gnome-system-monitor:4529): WARNING **: Failed to load icon for nicotine(4048) : Error opening file: No such file or directory
terminate called after throwing an instance of 'Gio::Error'

Program received signal SIGABRT, Aborted.
0x00007ffff3c5e3a5 in __GI_raise (sig=6) at ../nptl/sysdeps/unix/sysv/linux/raise.c:64
64	../nptl/sysdeps/unix/sysv/linux/raise.c: No such file or directory.
	in ../nptl/sysdeps/unix/sysv/linux/raise.c
(gdb) backtrace

+ Trace 228892

#0 __GI_raise
at ../nptl/sysdeps/unix/sysv/linux/raise.c line 64
#1 __GI_abort
at abort.c line 92
#2 __gnu_cxx::__verbose_terminate_handler()
from /usr/lib/x86_64-linux-gnu/libstdc++.so.6
#3 ??
from /usr/lib/x86_64-linux-gnu/libstdc++.so.6
#4 std::terminate()
from /usr/lib/x86_64-linux-gnu/libstdc++.so.6
#5 __cxa_throw
from /usr/lib/x86_64-linux-gnu/libstdc++.so.6
#6 Gio::Error::throw_func(_GError*)
from /usr/lib/libgiomm-2.4.so.1
#7 Glib::Error::throw_exception(_GError*)
from /usr/lib/libglibmm-2.4.so.1
#8 Gtk::IconTheme::load_icon(Glib::ustring const&, int, Gtk::IconLookupFlags) const
from /usr/lib/libgtkmm-3.0.so.1
#9 ??
#10 ??
#11 g_closure_invoke
from /usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0
#12 ??
from /usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0
#13 g_signal_emit_valist
from /usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0
#14 g_signal_emit
from /usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0
#15 ??
from /usr/lib/libwnck-3.so.0
#16 ??
from /usr/lib/libwnck-3.so.0
#17 g_main_context_dispatch
from /lib/x86_64-linux-gnu/libglib-2.0.so.0
#18 ??
from /lib/x86_64-linux-gnu/libglib-2.0.so.0
#19 g_main_context_iteration
from /lib/x86_64-linux-gnu/libglib-2.0.so.0
#20 gtk_main_iteration
from /usr/lib/libgtk-3.so.0
#21 ??
from /usr/lib/libwnck-3.so.0
#22 ??
from /usr/lib/libwnck-3.so.0
#23 g_main_context_dispatch
from /lib/x86_64-linux-gnu/libglib-2.0.so.0
#24 ??
from /lib/x86_64-linux-gnu/libglib-2.0.so.0
#25 g_main_loop_run
from /lib/x86_64-linux-gnu/libglib-2.0.so.0
#26 gtk_main
from /usr/lib/libgtk-3.so.0
#27 ??
#28 __libc_start_main
at libc-start.c line 226
#29 ??
#30 ??
#31 ??
#32 ??
#33 ??
#34 ??


--

Ubuntu 11.10
nicotine+             1.2.16+dfsg-1
gnome-system-monitor  3.2.0-0ubuntu1

Comment 1 André Klapper 2011-10-22 23:58:44 UTC

Thanks for taking the time to report this bug.

What is nicotine+?
Did you report this to Ubuntu's bugtracker?

Also, unfortunately, that stack trace is missing some elements that will help a lot to solve the problem, so it will be hard for the developers to fix that crash. Can you get us a stack trace with debugging symbols? Please see http://live.gnome.org/GettingTraces for more information on how to do so and attach it to this bug report. Thanks in advance!

Comment 2 Guillaume Mureu 2011-10-23 12:47:33 UTC

Nicotine+ is a p2p client for the soulseek network :
http://nicotine-plus.sourceforge.net/

I didn't report to Ubuntu, should I ?

Here's the complete trace:

+ Trace 228896

#0 __GI_raise
at ../nptl/sysdeps/unix/sysv/linux/raise.c line 64
#1 __GI_abort
at abort.c line 92
#2 __gnu_cxx::__verbose_terminate_handler
at ../../../../src/libstdc++-v3/libsupc++/vterminate.cc line 95
#3 __cxxabiv1::__terminate
at ../../../../src/libstdc++-v3/libsupc++/eh_terminate.cc line 40
#4 std::terminate
at ../../../../src/libstdc++-v3/libsupc++/eh_terminate.cc line 50
#5 __cxxabiv1::__cxa_throw
at ../../../../src/libstdc++-v3/libsupc++/eh_throw.cc line 83
#6 Gio::Error::throw_func
at error.cc line 55
#7 Glib::Error::throw_exception
at error.cc line 167
#8 Gtk::IconTheme::load_icon(Glib::ustring const&, int, Gtk::IconLookupFlags) const
from /usr/lib/libgtkmm-3.0.so.1
#9 procman::IconThemeWrapper::load_icon
at iconthemewrapper.cpp line 14
#10 PrettyTable::on_application_opened
at prettytable.cpp line 55
#11 g_closure_invoke
at /build/buildd/glib2.0-2.30.0/./gobject/gclosure.c line 774
#12 signal_emit_unlocked_R
at /build/buildd/glib2.0-2.30.0/./gobject/gsignal.c line 3272
#13 g_signal_emit_valist
at /build/buildd/glib2.0-2.30.0/./gobject/gsignal.c line 3003
#14 g_signal_emit
at /build/buildd/glib2.0-2.30.0/./gobject/gsignal.c line 3060
#15 emit_application_opened
at screen.c line 2363
#16 update_client_list
at screen.c line 1655
#17 do_update_now
at screen.c line 2251
#18 update_idle
at screen.c line 2274
#19 g_main_dispatch
at /build/buildd/glib2.0-2.30.0/./glib/gmain.c line 2441
#20 g_main_context_dispatch
at /build/buildd/glib2.0-2.30.0/./glib/gmain.c line 3011
#21 g_main_context_iterate
at /build/buildd/glib2.0-2.30.0/./glib/gmain.c line 3089
#22 g_main_context_iteration
at /build/buildd/glib2.0-2.30.0/./glib/gmain.c line 3152
#23 gtk_main_iteration
from /usr/lib/libgtk-3.so.0
#24 wnck_find_pid_for_resource_r
at util.c line 311
#25 wnck_pid_read_resource_usage_fill_cache
at util.c line 415
#26 g_main_dispatch
at /build/buildd/glib2.0-2.30.0/./glib/gmain.c line 2441
#27 g_main_context_dispatch
at /build/buildd/glib2.0-2.30.0/./glib/gmain.c line 3011
#28 g_main_context_iterate
at /build/buildd/glib2.0-2.30.0/./glib/gmain.c line 3089
#29 g_main_loop_run
at /build/buildd/glib2.0-2.30.0/./glib/gmain.c line 3297
#30 gtk_main
from /usr/lib/libgtk-3.so.0
#31 main
at procman.cpp line 719

Comment 3 Guillaume Mureu 2011-10-23 16:50:39 UTC

The full trace:

+ Trace 228897

#0 __GI_raise
at ../nptl/sysdeps/unix/sysv/linux/raise.c line 64
#1 __GI_abort
at abort.c line 92
#2 __gnu_cxx::__verbose_terminate_handler
at ../../../../src/libstdc++-v3/libsupc++/vterminate.cc line 95
#3 __cxxabiv1::__terminate
at ../../../../src/libstdc++-v3/libsupc++/eh_terminate.cc line 40
#4 std::terminate
at ../../../../src/libstdc++-v3/libsupc++/eh_terminate.cc line 50
#5 __cxxabiv1::__cxa_throw
at ../../../../src/libstdc++-v3/libsupc++/eh_throw.cc line 83
#6 Gio::Error::throw_func
at error.cc line 55
#7 Glib::Error::throw_exception
at error.cc line 167
#8 Gtk::IconTheme::load_icon
at icontheme.cc line 323
#9 procman::IconThemeWrapper::load_icon
at iconthemewrapper.cpp line 14
#10 PrettyTable::on_application_opened
at prettytable.cpp line 55
#11 g_closure_invoke
at /build/buildd/glib2.0-2.30.0/./gobject/gclosure.c line 774
#12 signal_emit_unlocked_R
at /build/buildd/glib2.0-2.30.0/./gobject/gsignal.c line 3272
#13 g_signal_emit_valist
at /build/buildd/glib2.0-2.30.0/./gobject/gsignal.c line 3003
#14 g_signal_emit
at /build/buildd/glib2.0-2.30.0/./gobject/gsignal.c line 3060
#15 emit_application_opened
at screen.c line 2363
#16 update_client_list
at screen.c line 1655
#17 do_update_now
at screen.c line 2251
#18 update_idle
at screen.c line 2274
#19 g_main_dispatch
at /build/buildd/glib2.0-2.30.0/./glib/gmain.c line 2441
#20 g_main_context_dispatch
at /build/buildd/glib2.0-2.30.0/./glib/gmain.c line 3011
#21 g_main_context_iterate
at /build/buildd/glib2.0-2.30.0/./glib/gmain.c line 3089
#22 g_main_context_iteration
at /build/buildd/glib2.0-2.30.0/./glib/gmain.c line 3152
#23 gtk_main_iteration
at /build/buildd/gtk+3.0-3.2.0/./gtk/gtkmain.c line 1467
#24 wnck_find_pid_for_resource_r
at util.c line 311
#25 wnck_pid_read_resource_usage_fill_cache
at util.c line 415
#26 g_main_dispatch
at /build/buildd/glib2.0-2.30.0/./glib/gmain.c line 2441
#27 g_main_context_dispatch
at /build/buildd/glib2.0-2.30.0/./glib/gmain.c line 3011
#28 g_main_context_iterate
at /build/buildd/glib2.0-2.30.0/./glib/gmain.c line 3089
#29 g_main_loop_run
at /build/buildd/glib2.0-2.30.0/./glib/gmain.c line 3297
#30 gtk_main
at /build/buildd/gtk+3.0-3.2.0/./gtk/gtkmain.c line 1367
#31 main
at procman.cpp line 719

Comment 4 Robert Roth 2012-01-03 22:00:09 UTC

Created attachment 204531 [details] [review]
Proposed patch

The proposed patch adds an additional catch in the iconthemewrapper's load icon to avoid crash if we can't load the icon because of file not found or other gio errors.

Comment 5 Robert Roth 2012-01-03 22:18:01 UTC

Created attachment 204534 [details] [review]
Proposed patch v2

Fixed patch by fixing debug message parameter list to match the template string.

Comment 6 Chris Kühl 2012-01-08 21:46:05 UTC

Looks fine. Do you have access to push that Robert? If so, please do.

Comment 7 Robert Roth 2012-01-08 22:13:43 UTC

No, I don't have commit access, so please push it.

Comment 8 Chris Kühl 2012-01-08 22:22:01 UTC

Ok, I'll push. Please apply for GNOME git access. I can vouch for you.

Comment 9 Chris Kühl 2012-01-08 22:26:25 UTC

Fix can be found in commit 0d394b2d9df72d9ad67deafebf9174b38b42a6d9.