diff options
| author | Lukas Larsson <[email protected]> | 2014-11-05 15:27:06 +0100 |
|---|---|---|
| committer | Lukas Larsson <[email protected]> | 2014-11-05 15:27:06 +0100 |
| commit | 7513bb7670cf7081c4b70e240f99f9424b612f0b (patch) | |
| tree | bc9aabb5e925541f41301a93d40bf11b00bfe533 /erts/emulator/beam/bif.c | |
| parent | 965187787f603181aadbe5effdbb8fa6dc90b9ac (diff) | |
| parent | 16d8a6ce56fd066efa9ecb1d3fa3b3dab6d9613c (diff) | |
| download | otp-7513bb7670cf7081c4b70e240f99f9424b612f0b.tar.gz otp-7513bb7670cf7081c4b70e240f99f9424b612f0b.tar.bz2 otp-7513bb7670cf7081c4b70e240f99f9424b612f0b.zip | |
Merge branch 'lukas/erts/fix_undefined_behaviour/OTP-12290' into maint
* lukas/erts/fix_undefined_behaviour/OTP-12290:
erts: Fix ub in list_to_integer and bignum div
Diffstat (limited to 'erts/emulator/beam/bif.c')
| -rw-r--r-- | erts/emulator/beam/bif.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/erts/emulator/beam/bif.c b/erts/emulator/beam/bif.c index a5be8e1529..42dd160e38 100644 --- a/erts/emulator/beam/bif.c +++ b/erts/emulator/beam/bif.c @@ -2772,6 +2772,7 @@ static int do_list_to_integer(Process *p, Eterm orig_list, Eterm *integer, Eterm *rest) { Sint i = 0; + Uint ui = 0; int skip = 0; int neg = 0; int n = 0; @@ -2825,8 +2826,8 @@ static int do_list_to_integer(Process *p, Eterm orig_list, unsigned_val(CAR(list_val(lst))) > '9') { break; } - i = i * 10; - i = i + unsigned_val(CAR(list_val(lst))) - '0'; + ui = ui * 10; + ui = ui + unsigned_val(CAR(list_val(lst))) - '0'; n++; lst = CDR(list_val(lst)); if (is_nil(lst)) { @@ -2850,7 +2851,8 @@ static int do_list_to_integer(Process *p, Eterm orig_list, */ if (n <= SMALL_DIGITS) { /* It must be small */ - if (neg) i = -i; + if (neg) i = -(Sint)ui; + else i = (Sint)ui; res = make_small(i); } else { lg2 = (n+1)*230/69+1; |