Giters

Manuel-Kehl / Go-For-It

A stylish to-do list with built-in productivity timer. NOTE: all contributions should go to Jonathan Moerman's fork because this is where active development is taking place: https://github.com/JMoerman/Go-For-It

Home Page:http://manuel-kehl.de/projects/go-for-it/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Crashes when there's a double space before project tag

daniellandau opened this issue · comments

commented

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.
commented

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.