pgsql: Add safeguards in LSN, numeric and float calculation for custom

From: Michael Paquier <michael(at)paquier(dot)xyz>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Add safeguards in LSN, numeric and float calculation for custom
Date: 2019-08-05 06:36:11
Message-ID: E1huWbL-00039x-Mx@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Add safeguards in LSN, numeric and float calculation for custom errors

Those data types use parsing and/or calculation wrapper routines which
can generate some generic error messages in the event of a failure. The
caller of these routines can also pass a pointer variable settable by
the routine to track if an error has happened, letting the caller decide
what to do in the event of an error and what error message to generate.

Those routines have been slacking the initialization of the tracking
flag, which can be confusing when reading the code, so add some
safeguards against calls of these parsing routines which could lead to a
dubious result.

The LSN parsing gains an assertion to make sure that the tracking flag
is set, while numeric and float paths initialize the flag to a saner
state.

Author: Jeevan Ladhe
Reviewed-by: Álvaro Herrera, Michael Paquier
Discussion: https://postgr.es/m/CAOgcT0NOM9oR0Hag_3VpyW0uF3iCU=BDUFSPfk9JrWXRcWQHqw@mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/a76cfba663ceab79b891cc81a5f941051755b3b0

Modified Files
--------------
src/backend/utils/adt/float.c | 20 +++++++++++++-------
src/backend/utils/adt/numeric.c | 12 ++++++++++++
src/backend/utils/adt/pg_lsn.c | 3 +++
3 files changed, 28 insertions(+), 7 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Peter Eisentraut 2019-08-05 14:04:51 pgsql: Translation updates
Previous Message Michael Paquier 2019-08-05 05:31:30 pgsql: Fix tab completion for ALTER LANGUAGE in psql