diff options
| author | Dean Rasheed <dean.a.rasheed@gmail.com> | 2024-08-26 11:00:20 +0100 |
|---|---|---|
| committer | Dean Rasheed <dean.a.rasheed@gmail.com> | 2024-08-26 11:00:20 +0100 |
| commit | 7cac6307a43c1fcbd050724d3995a489da62e852 (patch) | |
| tree | cd1f77a6746883cd5e5a031a7e0ecdfaef5c8d6e /src/backend/utils/adt/numeric.c | |
| parent | 8daa62a10c911c851f7e9ec5ef7b90cfd4b73212 (diff) | |
| download | postgresql-7cac6307a43c1fcbd050724d3995a489da62e852.tar.gz postgresql-7cac6307a43c1fcbd050724d3995a489da62e852.zip | |
Fix compiler warning in mul_var_short().
Some compilers (e.g., gcc before version 7) mistakenly think "carry"
might be used uninitialized.
Reported by Tom Lane, per various buildfarm members, e.g. arowana.
Diffstat (limited to 'src/backend/utils/adt/numeric.c')
| -rw-r--r-- | src/backend/utils/adt/numeric.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/backend/utils/adt/numeric.c b/src/backend/utils/adt/numeric.c index 44d88e90079..15b517ba988 100644 --- a/src/backend/utils/adt/numeric.c +++ b/src/backend/utils/adt/numeric.c @@ -8969,7 +8969,7 @@ mul_var_short(const NumericVar *var1, const NumericVar *var2, int res_ndigits; NumericDigit *res_buf; NumericDigit *res_digits; - uint32 carry; + uint32 carry = 0; uint32 term; /* Check preconditions */ @@ -9018,7 +9018,6 @@ mul_var_short(const NumericVar *var1, const NumericVar *var2, * res_ndigits = var2ndigits + 1 * ---------- */ - carry = 0; for (int i = var2ndigits - 1; i >= 0; i--) { term = PRODSUM1(var1digits, 0, var2digits, i) + carry; |