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 586541 - Crashes on hitting enter twice before URLs
Crashes on hitting enter twice before URLs
Status: RESOLVED FIXED
Product: gnote
Classification: Applications
Component: main
0.5.x
Other Linux
: Normal critical
: 1.0
Assigned To: gnote-maint
gnote-maint
: 599786 632797 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2009-06-21 15:47 UTC by Artiom MOLCHANOV
Modified: 2010-10-21 16:05 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
This is the note I try to edit (2.32 KB, text/plain)
2009-06-21 15:48 UTC, Artiom MOLCHANOV
  Details
.note file (1.45 KB, application/octet-stream)
2009-06-24 09:14 UTC, Artiom MOLCHANOV
  Details
backtrace (3.20 KB, text/plain)
2009-07-20 09:03 UTC, Jani Monoses
  Details
gnote crash output (1.22 KB, text/plain)
2009-11-05 20:40 UTC, mohamedhagag1981
  Details
Proposed fix (1.21 KB, patch)
2009-11-26 01:36 UTC, Debarshi Ray
accepted-commit_now Details | Review

Description Artiom MOLCHANOV 2009-06-21 15:47:34 UTC
A have a note with URLs and then I try to split the second URL in this note (put the cursor inside the URL and type ENTER) it crashes:

Distribution: Unknown
Gnome Release: 2.26.2 2009-05-21 (Archlinux)
BugBuddy Version: 2.26.0

System: Linux 2.6.29-ARCH #1 SMP PREEMPT Wed May 20 07:06:02 UTC 2009 i686
X Vendor: The X.Org Foundation
X Vendor Release: 10601901
Selinux: No
Accessibility: Disabled
GTK+ Theme: infinity
Icon Theme: infinity
GTK+ Modules: canberra-gtk-module, gnomebreakpad

Memory status: size: 0 vsize: 0 resident: 0 share: 0 rss: 0 rss_rlim: 0
CPU usage: start_time: 0 rtime: 0 utime: 0 stime: 0 cutime:0 cstime: 0 timeout: 0 it_real_value: 0 frequency: 0



---- Critical and fatal warnings logged during execution ----

** GLib-GObject **: g_signal_emit_valist: assertion `G_TYPE_CHECK_INSTANCE (instance)' failed 
** GLib-GObject **: g_signal_handlers_destroy: assertion `G_TYPE_CHECK_INSTANCE (instance)' failed 
** GLib-GObject **: g_signal_handlers_destroy: assertion `G_TYPE_CHECK_INSTANCE (instance)' failed 
** GLib-GObject **: g_type_class_peek_parent: assertion `g_class != NULL' failed 


----------- .xsession-errors ---------------------
(gnote:10258): GLib-GObject-CRITICAL **: g_type_class_peek_parent: assertion `g_class != NULL' failed
(gnote:10275): GLib-GObject-WARNING **: instance with invalid (NULL) class pointer
(gnote:10275): GLib-GObject-CRITICAL **: g_signal_emit_valist: assertion `G_TYPE_CHECK_INSTANCE (instance)' failed
(gnote:10275): GLib-GObject-WARNING **: instance with invalid (NULL) class pointer
(gnote:10275): GLib-GObject-CRITICAL **: g_signal_handlers_destroy: assertion `G_TYPE_CHECK_INSTANCE (instance)' failed
(gnote:10275): GLib-GObject-WARNING **: instance with invalid (NULL) class pointer
(gnote:10275): GLib-GObject-CRITICAL **: g_signal_handlers_destroy: assertion `G_TYPE_CHECK_INSTANCE (instance)' failed
(gnote:10275): GLib-GObject-CRITICAL **: g_type_class_peek_parent: assertion `g_class != NULL' failed
--------------------------------------------------
Comment 1 Artiom MOLCHANOV 2009-06-21 15:48:15 UTC
Created attachment 137122 [details]
This is the note I try to edit
Comment 2 Hubert Figuiere (:hub) 2009-06-22 05:09:52 UTC
(In reply to comment #1)
> Created an attachment (id=137122) [edit]
> This is the note I try to edit
> 

It would be nice to actually get the .note file, not an XML export. Thanks.
Comment 3 Hubert Figuiere (:hub) 2009-06-22 05:37:29 UTC
(In reply to comment #2)

> It would be nice to actually get the .note file, not an XML export. Thanks.

I meant "not an HTML export."

Comment 4 Artiom MOLCHANOV 2009-06-24 09:14:41 UTC
Created attachment 137297 [details]
.note file
Comment 5 Alessio "Spinus" Moscatello 2009-07-04 13:17:50 UTC
I've noticed that this crash is trigged even just moving the url in a new row.

Step to reproduce:
*enter an url
*position the cursor at the beginning of the url
*press two times return

The first time you press return the url is correctly moved in a new row, but the second time Gnote crashes

Output:

(gnote:22936): GLib-GObject-WARNING **: instance with invalid (NULL) class pointer

(gnote:22936): GLib-GObject-CRITICAL **: g_signal_emit_valist: assertion `G_TYPE_CHECK_INSTANCE (instance)' failed

(gnote:22936): GLib-GObject-WARNING **: instance with invalid (NULL) class pointer

(gnote:22936): GLib-GObject-CRITICAL **: g_signal_handlers_destroy: assertion `G_TYPE_CHECK_INSTANCE (instance)' failed

(gnote:22936): GLib-GObject-WARNING **: instance with invalid (NULL) class pointer

(gnote:22936): GLib-GObject-CRITICAL **: g_signal_handlers_destroy: assertion `G_TYPE_CHECK_INSTANCE (instance)' failed

(gnote:22936): GLib-GObject-CRITICAL **: g_type_class_peek_parent: assertion `g_class != NULL' failed
Segmentation fault
Comment 6 Jani Monoses 2009-07-20 09:03:36 UTC
Created attachment 138783 [details]
backtrace

as noted in a previous comment this happens with twice moving an URL line (http://a or ftp://b tested) down a line by pressing return.
Comment 7 Hubert Figuiere (:hub) 2009-11-05 18:03:36 UTC
*** Bug 599786 has been marked as a duplicate of this bug. ***
Comment 8 mohamedhagag1981 2009-11-05 20:40:12 UTC
Created attachment 147036 [details]
gnote crash output

the problem occurs when i use keyboard to navigate through the Notes [ any linked notes ]
Comment 9 mohamedhagag1981 2009-11-19 17:49:37 UTC
any update about this bug ??
Comment 10 Debarshi Ray 2009-11-26 01:36:34 UTC
Created attachment 148497 [details] [review]
Proposed fix

The NoteEditor is Gtk::managed and constructing a Glib::RefPtr<NoteEditor> from a NoteEditor* does not increment the reference count. So the destruction of the Glib::RefPtr should be prevented from reducing its reference count.
Comment 11 Hubert Figuiere (:hub) 2009-11-26 04:42:34 UTC
Comment on attachment 148497 [details] [review]
Proposed fix

This patch looks ok to me. You can commit it if you want.
Comment 12 Debarshi Ray 2009-11-26 12:39:37 UTC
Committed and pushed.
Comment 13 Hubert Figuiere (:hub) 2010-10-21 16:05:52 UTC
*** Bug 632797 has been marked as a duplicate of this bug. ***