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 541313 - Crash on saving file
Crash on saving file
Status: RESOLVED FIXED
Product: anjuta
Classification: Applications
Component: plugins: editor: scintilla
SVN TRUNK
Other All
: Normal critical
: ---
Assigned To: Sébastien Granjoux
Anjuta maintainers
Depends on:
Blocks:
 
 
Reported: 2008-07-02 21:05 UTC by Yuriy Penkin
Modified: 2008-07-04 18:23 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
not completed fix (1.67 KB, patch)
2008-07-02 22:09 UTC, Yuriy Penkin 		none Details | Review

Description Yuriy Penkin 2008-07-02 21:05:58 UTC 
Steps to reproduce:
1. Open document
2. Type something
3. Save file


Stack trace:
(anjuta:26728): GLib-GIO-CRITICAL **: g_file_new_for_uri: assertion `uri != NULL' failed

(anjuta:26728): GLib-GIO-CRITICAL **: g_file_move: assertion `G_IS_FILE (destination)' failed

Program received signal SIGSEGV, Segmentation fault.

+ Trace 201937

Thread 3066496800 (LWP 26728)

  • #0 text_editor_save_file
    at text_editor.c line 1621
  • #1 isaveable_save
    at text_editor.c line 2753
  • #2 ianjuta_file_savable_save
    at ianjuta-file-savable.c line 81
  • #3 anjuta_docman_save_document
    at anjuta-docman.c line 708
  • #4 on_save_activate
    at action-callbacks.c line 111
  • #5 g_cclosure_marshal_VOID__VOID
    from /usr/lib/libgobject-2.0.so.0
  • #6 g_closure_invoke
    from /usr/lib/libgobject-2.0.so.0
  • #7 ??
    from /usr/lib/libgobject-2.0.so.0
  • #8 g_signal_emit_valist
    from /usr/lib/libgobject-2.0.so.0
  • #9 g_signal_emit
    from /usr/lib/libgobject-2.0.so.0
  • #10 ??
    from /usr/lib/libgtk-x11-2.0.so.0
  • #11 gtk_action_activate
    from /usr/lib/libgtk-x11-2.0.so.0
  • #12 g_cclosure_marshal_VOID__VOID
    from /usr/lib/libgobject-2.0.so.0 

Other information:
Comment 1 Yuriy Penkin 2008-07-02 22:09:13 UTC 
Created attachment 113885 [details] [review]
not completed fix

Patch not completed yet. but resolve one reason of crash
Comment 2 Johannes Schmid 2008-07-02 22:38:36 UTC 
Hmm, I hardly use scintilla so I didn't test that much. Anyway, can you make sure to use some of the advanced gio funtionality that provide all the backup/symlink stuff already.

(see g_file_replace_contents for example).

Thanks!
Comment 3 Sébastien Granjoux 2008-07-04 18:00:37 UTC 
I'm taking care of this. Thanks for your patch Yuriy. I think the other thing missing is the change to adapt to the new saved signal argument.
Comment 4 Johannes Schmid 2008-07-04 18:23:47 UTC 
Just committed a fix to trunk. Gio does most of the work so we don't have to care about creating a backup file ourselves.

Thanks anyway for the patch!