Bug 358396 – crash at startup

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 358396 - crash at startup


Summary:	crash at startup


Status:	RESOLVED FIXED

Product:	rhythmbox
Classification:	Other
Component:	general
Version:	HEAD
Hardware:	Other Linux

Importance:	Normal normal
Target Milestone:	---
Assigned To:	RhythmBox Maintainers
QA Contact:	RhythmBox Maintainers

URL:
Whiteboard:

Depends on:
Blocks:

Reported:	2006-09-29 23:01 UTC by William Jon McCann
Modified:	2007-02-22 11:28 UTC

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

Attachments
patch (1.27 KB, patch) 2006-09-29 23:18 UTC, William Jon McCann	committed	Details \| Review

Description William Jon McCann 2006-09-29 23:01:23 UTC

Just updated rb from cvs for the first time in a week or two.

Program received signal SIGSEGV, Segmentation fault.
[Switching to Thread 83385248 (LWP 14471)]
0x080e52e5 in rhythmdb_tree_load (rdb=0x8a14018, die=0x8a140ac, 
    error=0x4f85378) at rhythmdb-tree.c:631
631             g_free (ctx);

+ Trace 73625

Thread 4 (Thread 83385248 (LWP 14471))

#0 rhythmdb_tree_load
at rhythmdb-tree.c line 631
#1 rhythmdb_load_thread_main
at rhythmdb.c line 2355
#2 g_thread_create_proxy
at gthread.c line 553
#3 start_thread
from /lib/libpthread.so.0
#4 clone
from /lib/libc.so.6

Thread 1 (Thread -1208473904 (LWP 14464))

#0 pango_layout_get_pixel_extents
at pango-layout.c line 2259
#1 get_size
at gtkcellrenderertext.c line 1520
#2 IA__gtk_cell_renderer_get_size
at gtkcellrenderer.c line 507
#3 IA__gtk_tree_view_column_cell_get_size
at gtktreeviewcolumn.c line 2578
#4 validate_row
at gtktreeview.c line 5554
#5 validate_visible_area
at gtktreeview.c line 5882
#6 do_presize_handler
at gtktreeview.c line 6223
#7 presize_handler_callback
at gtktreeview.c line 6247
#8 g_idle_dispatch
at gmain.c line 3926
#9 IA__g_main_context_dispatch
at gmain.c line 2045
#10 g_main_context_iterate
at gmain.c line 2677
#11 IA__g_main_loop_run
at gmain.c line 2881
#12 IA__gtk_main
at gtkmain.c line 1001
#13 main
at main.c line 372

Comment 1 William Jon McCann 2006-09-29 23:18:22 UTC

Created attachment 73679 [details] [review]
patch

* Can't use ctx here because it was freed already.
* Should define the GError locally since a NULL may be passed in for the GError pointer pointer.

Comment 2 Jonathan Matthew 2006-09-30 01:22:20 UTC

That looks more sensible than the change I made, so I committed it.  Does this actually fix the crash?

Comment 3 James "Doc" Livingston 2007-02-22 11:28:15 UTC

Assuming this is fixed, since we've had no new reports.