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 454037 - Invalid free when selecting Print in the menu
Invalid free when selecting Print in the menu
Status: RESOLVED FIXED
Product: gthumb
Classification: Other
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Paolo Bacchilega
Paolo Bacchilega
: 486823 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2007-07-05 18:18 UTC by Pascal Terjan
Modified: 2007-10-15 15:50 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch to use gtk_paper_size_free instead of g_free to free the GtkPaperSize (506 bytes, patch)
2007-07-05 18:59 UTC, Pascal Terjan
none Details | Review

Description Pascal Terjan 2007-07-05 18:18:07 UTC
No symbol table info available.
  • #1 raise
    from /lib/i686/libc.so.6
  • #2 abort
    from /lib/i686/libc.so.6
  • #3 __libc_message
    at ../sysdeps/unix/sysv/linux/libc_fatal.c line 170
  • #4 _int_free
    at malloc.c line 5617
  • #5 *__GI___libc_free
    at malloc.c line 3448
  • #6 IA__g_free
    at gmem.c line 187
  • #7 catalog_set_standard_page_size
    at print-callbacks.c line 1680
  • #8 print_catalog_dlg_full
    at print-callbacks.c line 2565
  • #9 print_catalog_dlg
    at print-callbacks.c line 2733
  • #10 gth_window_activate_action_file_print
    at gth-window-actions-callbacks.c line 259
  • #11 IA__g_cclosure_marshal_VOID__VOID
    at gmarshal.c line 77
  • #12 IA__g_closure_invoke
    at gclosure.c line 490
  • #13 signal_emit_unlocked_R
    at gsignal.c line 2440

Comment 1 Pascal Terjan 2007-07-05 18:20:59 UTC
==20585== 
==20585== Invalid free() / delete / delete[]
==20585==    at 0x401F0FF: free (in /usr/lib/valgrind/x86-linux/vgpreload_memcheck.so)
==20585==    by 0x4B8BF10: g_free (gmem.c:187)
==20585==    by 0x4093574: catalog_set_standard_page_size (print-callbacks.c:1680)
==20585==    by 0x4094FEC: print_catalog_dlg_full (print-callbacks.c:2565)
==20585==    by 0x40950F2: print_catalog_dlg (print-callbacks.c:2733)
==20585==    by 0x80A3F0D: gth_window_activate_action_file_print (gth-window-actions-callbacks.c:259)
==20585==    by 0x4B24E70: g_cclosure_marshal_VOID__VOID (gmarshal.c:77)
==20585==    by 0x4B16F5C: g_closure_invoke (gclosure.c:490)
==20585==    by 0x4B291FA: signal_emit_unlocked_R (gsignal.c:2440)
==20585==    by 0x4B2A664: g_signal_emit_valist (gsignal.c:2199)
==20585==    by 0x4B2A8EA: g_signal_emit (gsignal.c:2243)
==20585==    by 0x447CFB5: _gtk_action_emit_activate (gtkaction.c:880)
==20585==  Address 0x658D5B8 is 440 bytes inside a block of size 504 alloc'd
==20585==    at 0x401E710: memalign (in /usr/lib/valgrind/x86-linux/vgpreload_memcheck.so)
==20585==    by 0x401E76A: posix_memalign (in /usr/lib/valgrind/x86-linux/vgpreload_memcheck.so)
==20585==    by 0x4BA049E: slab_allocator_alloc_chunk (gslice.c:1115)
==20585==    by 0x4BA168D: g_slice_alloc (gslice.c:645)
==20585==    by 0x4BA17B5: g_slice_alloc0 (gslice.c:812)
==20585==    by 0x4B3873D: g_type_create_instance (gtype.c:1549)
==20585==    by 0x4B1D233: g_object_constructor (gobject.c:1046)
==20585==    by 0x4B1DA06: g_object_newv (gobject.c:937)
==20585==    by 0x4B1E534: g_object_new_valist (gobject.c:986)
==20585==    by 0x4B1E6F2: g_object_new (gobject.c:795)
==20585==    by 0x4A0C771: atk_no_op_object_new (in /usr/lib/libatk-1.0.so.0.1912.1)
==20585==    by 0x4A0C96A: (within /usr/lib/libatk-1.0.so.0.1912.1)
Comment 2 Pascal Terjan 2007-07-05 18:30:13 UTC
I suppose g_free at libgthumb/print-callbacks.c:1680 should be changed to gtk_paper_size_free
Comment 3 Pascal Terjan 2007-07-05 18:59:05 UTC
Created attachment 91267 [details] [review]
Patch to use gtk_paper_size_free instead of g_free to free the GtkPaperSize
Comment 4 Michael Chudobiak 2007-07-05 19:59:56 UTC
Thank you! Patch committed to gthumb-2-10 branch and trunk. It should appear in the next release of gThumb.

- Mike
Comment 5 Michael Chudobiak 2007-10-15 15:50:51 UTC
*** Bug 486823 has been marked as a duplicate of this bug. ***