shell/math: return string error indicator, not integer

function                                             old     new   delta
expand_and_evaluate_arith                             87     106     +19
expand_one_var                                      1563    1570      +7
arith                                                 12      18      +6
evaluate_string                                      678     680      +2
arith_apply                                         1269    1271      +2
builtin_umask                                        133     132      -1
ash_arith                                            118      75     -43
expand_vars_to_list                                 1094    1038     -56
------------------------------------------------------------------------------
(add/remove: 0/0 grow/shrink: 5/3 up/down: 36/-100)           Total: -64 bytes

Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>

1 files changed, 2 insertions, 9 deletions

diff --git a/shell/ash.c b/shell/ash.c
index c27ab7de2..ec887e088 100644
--- a/shell/ash.c
+++ b/shell/ash.c
@@ -5451,15 +5451,8 @@ ash_arith(const char *s)
 
         INT_OFF;
         result = arith(&math_state, s);
-        if (math_state.errcode < 0) {
-                if (math_state.errcode == -3)
-                        ash_msg_and_raise_error("exponent less than 0");
-                if (math_state.errcode == -2)
-                        ash_msg_and_raise_error("divide by zero");
-                if (math_state.errcode == -5)
-                        ash_msg_and_raise_error("expression recursion loop detected");
-                raise_error_syntax(s);
-        }
+        if (math_state.errmsg)
+                ash_msg_and_raise_error(math_state.errmsg);
         INT_ON;
 
         return result;