Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Special unicode emoji 👨‍💻 crashes Geany on backspace #2642

Closed
esotericpig opened this issue Nov 5, 2020 · 7 comments
Closed

Special unicode emoji 👨‍💻 crashes Geany on backspace #2642

esotericpig opened this issue Nov 5, 2020 · 7 comments
Labels
bug can't reproduce A developer couldn't reproduce the issue crash Geany crashes because of this issue

Comments

@esotericpig
Copy link

The "Man Technologist" 👨‍💻 (UTF-8) emoji crashes Geany on backspace.

To reproduce:

  1. Open Geany
  2. Paste 👨‍💻 into a new document and save as "bug.txt"
  3. Save the file
  4. Try to delete the emoji with backspace
  5. Should now crash

To fix:

  • Delete the character with another editor (I used Gedit)

Verbose messages:

(geany:1034609): dconf-DEBUG: 02:17:05.307: change_fast
(geany:1034609): dconf-DEBUG: 02:17:05.307: change_notify: /org/gtk/settings/file-chooser/window-position
Geany-INFO: 02:17:15.744: /home/name/Documents/bug.txt : None (UTF-8)
Geany-INFO: 02:17:15.746: /home/name/Documents/bug.txt : None (UTF-8)
(geany:1034609): dconf-DEBUG: 02:17:15.752: change_fast
(geany:1034609): dconf-DEBUG: 02:17:15.753: change_notify: /org/gtk/settings/file-chooser/
/usr/include/c++/10/bits/stl_vector.h:1063: std::vector<_Tp, _Alloc>::const_reference std::vector<_Tp, _Alloc>::operator[](std::vector<_Tp, _Alloc>::size_type) const [with _Tp = unsigned char; _Alloc = std::allocator<unsigned char>; std::vector<_Tp, _Alloc>::const_reference = const unsigned char&; std::vector<_Tp, _Alloc>::size_type = long unsigned int]: Assertion '__builtin_expect(__n < this->size(), true)' failed.
Aborted (core dumped)

Environment:

  • Geany 1.36 (built on 2020-05-24 with GTK 3.24.20, GLib 2.64.3)
  • Linux 5.8.14-200.fc32.x86_64 # 1 SMP Wed Oct 7 14:47:56 UTC 2020 x86_64 GNU/Linux
  • Fedora 32
@b4n
Copy link
Member

b4n commented Nov 5, 2020

I can't reproduce on Geany 1.37 (from January 14), fr_FR.utf8 / GTK 3.24.5, GLib 2.58.3 under Debian Buster, nor with latest Git.
What puzzles me a bit is that I don't see obvious changes that would have affected this. Could you try latest version (1.37) and see if it fixes it for you?

@b4n b4n added bug can't reproduce A developer couldn't reproduce the issue crash Geany crashes because of this issue labels Nov 5, 2020
@elextr
Copy link
Member

elextr commented Nov 5, 2020

WFM with Geany 1.36 (git >= 825acb2), en_AU.UTF-8 GTK 3.24.20, GLib 2.64.3 (825acb2 is 1.36.0 release tag)

@codebrainz
Copy link
Member

@esotericpig can you paste the full Help->Debug Messages?

@elextr
Copy link
Member

elextr commented Nov 6, 2020

@esotericpig make absolutely sure you are not running any plugins and re-test.

@esotericpig
Copy link
Author

I turned off all plugins, and it works fine.

I then turned on each plugin and found it to be the "Spell Check" plugin.

@esotericpig can you paste the full Help->Debug Messages?

It always crashes before anything useful, but here is it before the crash:

12:51:56: GLib-GIO DEBUG	: _g_io_module_get_default: Found default implementation gvfs (GDaemonVfs) for ‘gio-vfs’
12:51:56: Geany INFO		: Geany 1.36, en_US.UTF-8
12:51:56: Geany INFO		: GTK 3.24.23, GLib 2.64.6
12:51:56: Geany INFO		: System data dir: /usr/share/geany
12:51:56: Geany INFO		: User config dir: /home/name/.config/geany
12:51:56: Geany INFO		: Loaded GTK+ CSS theme '/usr/share/geany/geany.css'
12:51:56: Geany INFO		: Loaded GTK+ CSS theme '/usr/share/geany/geany-3.20.css'
12:51:56: Geany INFO		: System plugin path: /usr/lib64/geany
12:51:56: Geany INFO		: Added filetype CUDA (61).
12:51:56: Geany INFO		: Added filetype Kotlin (62).
12:51:56: Geany INFO		: Added filetype Cython (63).
12:51:56: Geany INFO		: Added filetype TypeScript (64).
12:51:56: Geany INFO		: Added filetype Arduino (65).
12:51:56: Geany INFO		: Added filetype Genie (66).
12:51:56: Geany INFO		: Added filetype Groovy (67).
12:51:56: Geany INFO		: Added filetype Clojure (68).
12:51:56: Geany INFO		: Added filetype Graphviz (69).
12:51:56: Geany INFO		: Added filetype Nim (70).
12:51:56: Geany INFO		: Added filetype Scala (71).
12:51:56: Geany INFO		: Added filetype JSON (72).
12:51:56: Geany INFO		: Added filetype Swift (73).
12:51:56: Geany INFO		: Loaded libvte from libvte-2.91.so.0
12:51:56: Geany INFO		: Loaded:   /usr/lib64/geany/addons.so (Addons)
12:51:56: Geany INFO		: Loaded:   /usr/lib64/geany/classbuilder.so (Class Builder)
12:51:56: SpellCheck DEBUG	: Initializing Enchant library version 2.2.12
12:51:56: GLib DEBUG	: unsetenv() is not thread-safe and should not be used after threads are created
12:51:56: Geany INFO		: Loaded:   /usr/lib64/geany/spellcheck.so (Spell Check)
12:51:56: Geany INFO		: /home/name/Documents/bug.txt : None (UTF-8)

@esotericpig
Copy link
Author

Since it's the Spell Check plugin, should I close this issue and open one up in Geany-plugins?

@elextr
Copy link
Member

elextr commented Nov 6, 2020

should I close this issue and open one up in Geany-plugins?

Yes thanks.

@elextr elextr closed this as completed Nov 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug can't reproduce A developer couldn't reproduce the issue crash Geany crashes because of this issue
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@codebrainz @b4n @elextr @esotericpig