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 112874 - gnome-sound-properties crashes [compare_func]
gnome-sound-properties crashes [compare_func]
Status: RESOLVED FIXED
Product: gnome-control-center
Classification: Core
Component: Sound
2.3.x
Other Linux
: High critical
: ---
Assigned To: Control-Center Maintainers
Control-Center Maintainers
: 113617 114494 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2003-05-13 03:47 UTC by Samuel Stringham
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
one line patch (594 bytes, patch)
2003-05-13 03:47 UTC, Samuel Stringham
none Details | Review
patch to fix network proxy preferences (1.64 KB, patch)
2003-07-02 13:12 UTC, Kjartan Maraas
none Details | Review

Description Samuel Stringham 2003-05-13 03:47:21 UTC
in compare_func, patch coming shortly.
Comment 1 Samuel Stringham 2003-05-13 03:47:58 UTC
Created attachment 16477 [details] [review]
one line patch
Comment 2 Samuel Stringham 2003-05-13 03:48:29 UTC
setting priority/severity appropriately.
Comment 3 Elijah Newren 2003-05-13 05:39:32 UTC
Adding PATCH and bugsquad keywords.
Comment 4 Andrew Sobala 2003-05-13 15:18:06 UTC
Samuel: can you paste a backtrace into the bug, please, so we can
catch duplicates.
Comment 5 Samuel Stringham 2003-05-13 17:03:52 UTC


  • #0 g_ascii_strcasecmp
    at gstrfuncs.c line 1735
  • #1 compare_func
    at sound-view.c line 84
  • #2 gtk_tree_model_sort_level_find_insert
    at gtktreemodelsort.c line 1711
  • #3 gtk_tree_model_sort_insert_value
    at gtktreemodelsort.c line 1754
  • #4 gtk_tree_model_sort_row_inserted
    at gtktreemodelsort.c line 686
  • #5 _gtk_marshal_VOID__BOXED_BOXED
    at gtkmarshalers.c line 862
  • #6 g_closure_invoke
    at gclosure.c line 437
  • #7 signal_emit_unlocked_R
    at gsignal.c line 2822
  • #8 g_signal_emit_valist
    at gsignal.c line 2554
  • #9 g_signal_emit
    at gsignal.c line 2612
  • #10 gtk_tree_model_row_inserted
    at gtktreemodel.c line 1208
  • #11 gtk_tree_store_insert_before
    at gtktreestore.c line 1221
  • #12 gtk_tree_store_append
    at gtktreestore.c line 1407
  • #13 foreach_cb
    at sound-view.c line 334
  • #14 category_cb
    at sound-properties.c line 602
  • #15 g_hash_table_foreach
    at ghash.c line 559
  • #16 sound_properties_category_foreach
    at sound-properties.c line 617
  • #17 sound_view_reload
    at sound-view.c line 377
  • #18 sound_view_new
    at sound-view.c line 313
  • #19 create_dialog
    at sound-properties-capplet.c line 74
  • #20 main
    at sound-properties-capplet.c line 171
  • #21 __libc_start_main
    from /lib/libc.so.6

Comment 6 Elijah Newren 2003-05-25 02:59:36 UTC
*** Bug 113617 has been marked as a duplicate of this bug. ***
Comment 7 Kjartan Maraas 2003-06-02 16:54:48 UTC
This goes for 2.2.x too?
Comment 8 Elijah Newren 2003-06-12 02:53:05 UTC
*** Bug 114494 has been marked as a duplicate of this bug. ***
Comment 9 Kjartan Maraas 2003-07-02 13:11:31 UTC
I think there are two bugs here. And possibly a third related one at
#109323.

Anyway, I got a patch from anders to fix the issue in #113617 and I'll
attach it here so people can try it out.
Comment 10 Kjartan Maraas 2003-07-02 13:12:11 UTC
Created attachment 17977 [details] [review]
patch to fix network proxy preferences
Comment 11 Kjartan Maraas 2003-07-02 13:13:32 UTC
Disregard the last one. I must be asleep. gnome-sound-properties is
clearly not the same as gnome-network-preferences :)
Comment 12 Christophe Fergeau 2003-07-18 15:34:44 UTC
Samuel, is it normal behaviour to get NULL strings in compare_func, or
is it hiding a bug somewhere else ? If it is normal, the first patch
looks good
Comment 13 Jonathan Blandford 2003-07-31 17:50:23 UTC
This was fixed by 2.3.3 with a different patch.

cmp = g_ascii_strcasecmp (str_a?str_a:"", str_b?str_b:"");

was done instead, as you need a reasonable value for cmp.