diff options
author | Alexis Hildebrandt <afh@surryhill.net> | 2014-12-16 19:35:36 +0100 |
---|---|---|
committer | Alexis Hildebrandt <afh@surryhill.net> | 2014-12-16 19:37:37 +0100 |
commit | 0a0f2f0d69e694dd7dd3489bdfe6a3ad6281c230 (patch) | |
tree | 59e926d974cabe8485273ed3cfa9691e6fc9c64c | |
parent | 5717a03e3b23a98873e83b3f00b855cfb8201d3a (diff) | |
download | fork-ledger-0a0f2f0d69e694dd7dd3489bdfe6a3ad6281c230.tar.gz fork-ledger-0a0f2f0d69e694dd7dd3489bdfe6a3ad6281c230.tar.bz2 fork-ledger-0a0f2f0d69e694dd7dd3489bdfe6a3ad6281c230.zip |
Fix crash when using -M with empty result
ledger -f /dev/null reg -M test causes a segmentation fault,
see bug 730 and duplicates 1080 and 1084 for details.
Kudos to Ikke for helping with debugging.
-rw-r--r-- | src/filters.cc | 2 | ||||
-rw-r--r-- | test/regress/730.test | 37 |
2 files changed, 38 insertions, 1 deletions
diff --git a/src/filters.cc b/src/filters.cc index bdc2983b..d24188f5 100644 --- a/src/filters.cc +++ b/src/filters.cc @@ -983,7 +983,7 @@ void interval_posts::flush() sort_posts_by_date()); // Determine the beginning interval by using the earliest post - if (all_posts.front() && + if (all_posts.size() > 0 && all_posts.front() && ! interval.find_period(all_posts.front()->date())) throw_(std::logic_error, _("Failed to find period for interval report")); diff --git a/test/regress/730.test b/test/regress/730.test new file mode 100644 index 00000000..d81a1c22 --- /dev/null +++ b/test/regress/730.test @@ -0,0 +1,37 @@ +; Using -M in combination with an empty result causes a segmentation fault +; therefore this test case does not have or need any test data + +test -f /dev/null -M reg +end test + +; Tests mentioned in #730 +test reg -M +end test + +test reg -M .foo +end test + +test reg -M -e 2012/01 +end test + + +; Tests mentioned in #1080 +test reg '^Expenses' and expr 'any(account =~ /^Assets:Cash/)' --period 'every week this month' +end test + +test bal '^Expenses' and expr 'any(account =~ /^Assets:Cash/)' --period 'every week this month' +end test + +test bal reg foo and expr 'any(account =~ /bar/)' --period 'every week' +end test + + +; Tests mentioned in #1084 +test b abc -M +end test + +test reg foo -M +end test + +test bal foo -M +end test |