summaryrefslogtreecommitdiffstats
path: root/bin/sh/arith_yacc.c
diff options
context:
space:
mode:
authorjilles <jilles@FreeBSD.org>2011-02-12 23:44:05 +0000
committerjilles <jilles@FreeBSD.org>2011-02-12 23:44:05 +0000
commit2fb060368627dc29279d5ac969e6c3eb6b0e7ea3 (patch)
treecc8681e662bed68e4af8833df6725164bd71882d /bin/sh/arith_yacc.c
parent4fe7a7870a6da7d72489d152af4b1a951eb50ae4 (diff)
downloadFreeBSD-src-2fb060368627dc29279d5ac969e6c3eb6b0e7ea3.zip
FreeBSD-src-2fb060368627dc29279d5ac969e6c3eb6b0e7ea3.tar.gz
sh: Detect dividing the smallest integer by -1.
This overflows and on some architectures such as amd64 it generates SIGFPE. Generate an error on all architectures.
Diffstat (limited to 'bin/sh/arith_yacc.c')
-rw-r--r--bin/sh/arith_yacc.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/bin/sh/arith_yacc.c b/bin/sh/arith_yacc.c
index 423e4e2..8aab8fb 100644
--- a/bin/sh/arith_yacc.c
+++ b/bin/sh/arith_yacc.c
@@ -125,6 +125,8 @@ static arith_t do_binop(int op, arith_t a, arith_t b)
case ARITH_DIV:
if (!b)
yyerror("division by zero");
+ if (a == ARITH_MIN && b == -1)
+ yyerror("divide error");
return op == ARITH_REM ? a % b : a / b;
case ARITH_MUL:
return a * b;
OpenPOWER on IntegriCloud