diff options
| author | Dean Rasheed <dean.a.rasheed@gmail.com> | 2021-01-05 11:07:12 +0000 |
|---|---|---|
| committer | Dean Rasheed <dean.a.rasheed@gmail.com> | 2021-01-05 11:07:12 +0000 |
| commit | 258b7700551c4fc01ecd55d7ead4085184d4dce3 (patch) | |
| tree | 746f76c610a5a60117fe7953cd260c7a18a02163 /src/backend/utils/adt/numeric.c | |
| parent | 7c98759e0485d26303f2572ce7724b6026fddd9c (diff) | |
| download | postgresql-258b7700551c4fc01ecd55d7ead4085184d4dce3.tar.gz postgresql-258b7700551c4fc01ecd55d7ead4085184d4dce3.zip | |
Fix numeric_power() when the exponent is INT_MIN.
In power_var_int(), the computation of the number of significant
digits to use in the computation used log(Abs(exp)), which isn't safe
because Abs(exp) returns INT_MIN when exp is INT_MIN. Use fabs()
instead of Abs(), so that the exponent is cast to a double before the
absolute value is taken.
Back-patch to 9.6, where this was introduced (by 7d9a4737c2).
Discussion: https://postgr.es/m/CAEZATCVd6pMkz=BrZEgBKyqqJrt2xghr=fNc8+Z=5xC6cgWrWA@mail.gmail.com
Diffstat (limited to 'src/backend/utils/adt/numeric.c')
| -rw-r--r-- | src/backend/utils/adt/numeric.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/backend/utils/adt/numeric.c b/src/backend/utils/adt/numeric.c index 73be69c740c..e347eaa885a 100644 --- a/src/backend/utils/adt/numeric.c +++ b/src/backend/utils/adt/numeric.c @@ -8601,7 +8601,7 @@ power_var_int(const NumericVar *base, int exp, NumericVar *result, int rscale) * to around log10(abs(exp)) digits, so work with this many extra digits * of precision (plus a few more for good measure). */ - sig_digits += (int) log(Abs(exp)) + 8; + sig_digits += (int) log(fabs(exp)) + 8; /* * Now we can proceed with the multiplications. |