Bug 336527 – crash on xcf.bz2 save

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 336527 - crash on xcf.bz2 save


Summary:	crash on xcf.bz2 save


Status:	RESOLVED FIXED

Product:	GIMP
Classification:	Other
Component:	General
Version:	git master
Hardware:	Other All

Importance:	Normal critical
Target Milestone:	2.4
Assigned To:	GIMP Bugs
QA Contact:	GIMP Bugs

URL:
Whiteboard:

Depends on:
Blocks:

Reported:	2006-03-29 18:21 UTC by Jakub Steiner
Modified:	2006-03-29 21:02 UTC

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

Description Jakub Steiner 2006-03-29 18:21:12 UTC

GIMP HEAD is crashing trying to save bzip2 compressed XCFs. Backtrace:

file_save_dialog_check_uri: URI = file:///home/jimmac/Desktop/22x22/stock-tool-measure_22.xcf.bz2
file_save_dialog_check_uri: basename = stock-tool-measure_22.xcf.bz2
file_save_dialog_check_uri: selected save_proc: NULL
file_save_dialog_check_uri: URI save_proc: bzip archive
file_save_dialog_check_uri: basename save_proc: bzip archive

file_save_dialog_check_uri: no save_proc was selected from the list
file_save_dialog_check_uri: use URI's proc 'bzip archive' so indirect saving works
gimp-2.3: fatal error: Segmentation fault
gimp-2.3 (pid:22331): [E]xit, [H]alt, show [S]tack trace or [P]roceed: s

+ Trace 67322

#0 __kernel_vsyscall
#1 __waitpid_nocancel
from /lib/tls/i686/cmov/libpthread.so.0
#2 g_on_error_stack_trace
from /usr/lib/libglib-2.0.so.0
#3 g_on_error_query
from /usr/lib/libglib-2.0.so.0
#4 gimp_eek
#5 gimp_fatal_error
#6 gimp_sigfatal_handler
at main.c line 579
#7 <signal handler called>
#8 strncmp
from /lib/tls/i686/cmov/libc.so.6
#9 file_proc_find_by_prefix
#10 file_utils_filename_to_uri
#11 file_save_invoker
#12 procedural_db_execute
#13 plug_in_handle_message
#14 plug_in_recv_message
#15 g_vasprintf
from /usr/lib/libglib-2.0.so.0
#16 g_main_context_dispatch
from /usr/lib/libglib-2.0.so.0
#17 g_main_context_check
from /usr/lib/libglib-2.0.so.0
#18 g_main_loop_run
from /usr/lib/libglib-2.0.so.0
#19 plug_in_run
#20 procedural_db_execute
#21 procedural_db_run_proc
#22 file_save
#23 file_save_dialog_response
#24 g_cclosure_marshal_VOID
from /usr/lib/libgobject-2.0.so.0
#25 g_closure_invoke
from /usr/lib/libgobject-2.0.so.0
#26 g_signal_stop_emission
from /usr/lib/libgobject-2.0.so.0
#27 g_signal_emit_valist
from /usr/lib/libgobject-2.0.so.0
#28 g_signal_emit
from /usr/lib/libgobject-2.0.so.0
#29 gtk_dialog_response
from /usr/lib/libgtk-x11-2.0.so.0
#30 gtk_dialog_response
from /usr/lib/libgtk-x11-2.0.so.0
#31 g_cclosure_marshal_VOID__VOID
#32 g_closure_invoke
from /usr/lib/libgobject-2.0.so.0
#33 g_signal_stop_emission
from /usr/lib/libgobject-2.0.so.0
#34 g_signal_emit_valist
from /usr/lib/libgobject-2.0.so.0
#35 g_signal_emit
from /usr/lib/libgobject-2.0.so.0
#36 gtk_button_clicked
from /usr/lib/libgtk-x11-2.0.so.0
#37 _gtk_button_set_depressed
#38 _gtk_button_set_depressed
#39 _gtk_marshal_BOOLEAN__BOXED
#40 g_cclosure_new_swap
from /usr/lib/libgobject-2.0.so.0
#41 g_closure_invoke
from /usr/lib/libgobject-2.0.so.0
#42 g_signal_stop_emission
from /usr/lib/libgobject-2.0.so.0
#43 g_signal_emit_valist
from /usr/lib/libgobject-2.0.so.0
#44 g_signal_emit
from /usr/lib/libgobject-2.0.so.0
#45 gtk_widget_activate
from /usr/lib/libgtk-x11-2.0.so.0
#46 gtk_propagate_event
from /usr/lib/libgtk-x11-2.0.so.0
#47 gtk_main_do_event
from /usr/lib/libgtk-x11-2.0.so.0
#48 _gdk_events_queue
from /usr/lib/libgdk-x11-2.0.so.0
#49 g_main_context_dispatch
from /usr/lib/libglib-2.0.so.0
#50 g_main_context_check
from /usr/lib/libglib-2.0.so.0
#51 g_main_loop_run
from /usr/lib/libglib-2.0.so.0
#52 app_run
#53 main
at main.c line 316

Comment 1 Michael Natterer 2006-03-29 21:02:42 UTC

Fixed in CVS:

2006-03-29  Michael Natterer  <mitch@gimp.org>

	* tools/pdbgen/pdb/fileops.pdb (file_save): pass the filename, not
	a GimpImage to file_utils_filename_to_uri(). Fixes bug #336527.

	* app/pdb/fileops_cmds.c: regenerated.