jeanguyomarch / eovim

The Enlightened Neovim
https://github.com/jeanguyomarch/eovim/wiki
MIT License
193 stars 3 forks source link

segfault: String would overflow textgrid. Truncating. #16

Closed wezm closed 6 years ago

wezm commented 6 years ago

I built and installed revision ce028d2 via the AUR package. When I run it the window appears for a second then it crashes with the following error:

ERR<29523>:eovim /home/wmoore/aur/eovim-git/src/eovim-git/src/termview.c:860 termview_put() String would overflow textgrid. Truncating.
## Copy & Paste the below (until EOF) into a terminal, then hit Enter

eina_btlog << EOF
/usr/lib/libeina.so.1    0x7f2d1ce1988e 0x7f2d1cdf3000
/usr/lib/libeina.so.1    0x7f2d1ce1a601 0x7f2d1cdf3000
/usr/lib/libeina.so.1    0x7f2d1ce1ba23 0x7f2d1cdf3000
[1]    29523 segmentation fault (core dumped)  eovim

The backtrace looks truncated but running eina_btlog on the frames shown gives:

    /usr/lib/libeina.so.1| ??/??: 0 @ eina_log_print_cb_journald()
    /usr/lib/libeina.so.1| ??/??: 0 @ eina_log_print_cb_stdout()
    /usr/lib/libeina.so.1| ??/??: 0 @ eina_log_print()

The same crash also happens when running with -N. When run with -g 80x24 it does not crash.

jeanguyomarch commented 6 years ago

Hi, I think there are two errors here:

I've already experienced the EFL bug, but haven't pinpoint the version it is present in. What version of EFL are you using? An easy way to circumvent this issue while waiting for a fix in the EFL would be to disable the backtrace feature by setting EINA_LOG_BACKTRACE to something like -1.

The resizing bug in Eovim is a real problem, I'll work on that.

jeanguyomarch commented 6 years ago

If you are using the GIT version of EFL, the segfaults should now be fixed.

wezm commented 6 years ago

Totally missed your original reply a week ago. Thanks for looking into a fix. I'll try it out when I'm back at work tomorrow.

wezm commented 6 years ago

Tried it out and it no longer crashes 🎉 Thanks.

jeanguyomarch commented 6 years ago

Nice :)