Bug #1048

- Fixed bug #1048, which segfaulted rather than complain about syntax (thanks
  to Bryce Harrington, Štěpán Henek).

AUTHORS

@@ -76,6 +76,7 @@ The following submitted code, packages or analysis, and deserve special thanks:
   Markus Kuhn
   Scott Kostyshak
   Erik Wenzel
+  Štěpán Henek
 
 Thanks to the following, who submitted detailed bug reports and excellent
 suggestions:

ChangeLog

@@ -17,6 +17,8 @@ Bugs
    external utilities on Solaris and NetBSD
  + Fixed bug #1044, where 'task projects' considers newly deleted tasks and
    provides an incorrect summary.
+ + Fixed bug #1048, which segfaulted rather than complain about syntax (thanks
+   to Bryce Harrington, Štěpán Henek).
 
 ------ old releases ------------------------------
 

src/E9.cpp

@@ -116,7 +116,7 @@ void E9::eval (const Task& task, std::vector <Arg>& value_stack)
 
       // TODO Not sure this is correct.
       // TODO No longer sure why I was unsure in the first place.
-      else if (arg->_raw == "-" && value_stack.size () < 2)
+      else if (arg->_raw == "-" && value_stack.size () == 1)
       {
         Arg right = value_stack.back ();
         value_stack.pop_back ();