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.

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

Crashes when there's a double space before project tag #127

Closed
daniellandau opened this issue Feb 25, 2018 · 1 comment
Closed

Crashes when there's a double space before project tag #127

daniellandau opened this issue Feb 25, 2018 · 1 comment

Comments

@daniellandau
Copy link
Contributor

Expected behavior

App should not crash with valid (or invalid for that matter) input.

Actual behavior

Outputs

load_task_stores
Reading file: /home/dlandau/Asiakirjat/todo/todo.txt
Reading file: /home/dlandau/Asiakirjat/todo/done.txt

and then exits with code 139. GUI never shows up. Journal has

kernel: de.manuel_kehl.[9846]: segfault at 72 ip 0000003153a80526 sp 00007fffb0bb0858 error 4 in libc-2.24.so[3153a00000+194000]

How to reproduce the behavior

Try to open the app with a todo.txt with the following contents (notice the double space before +tag)

2018-02-25 whatever  +tag

Operating System used

Ubuntu 16.04

Installation source

Installed from Flathub.

% flatpak run de.manuel_kehl.go-for-it  --version
Go For It! 1.6.3
Copyright 2014-2017 'Go For it!' Developers.
@JMoerman
Copy link
Collaborator

Thanks for reporting this issue!
This should be fixed with ed31d42.
This issue was not limited to tasks with tags, long-ish strings with double spaces would cause crashes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants