]> granicus.if.org Git - postgresql/commit
Avoid unexpected conversion overflow in planner for distant date values.
authorTom Lane <tgl@sss.pgh.pa.us>
Wed, 29 Dec 2010 03:49:57 +0000 (22:49 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Wed, 29 Dec 2010 03:50:51 +0000 (22:50 -0500)
commit1fa37ac25d6aa26443e47f96aafe87c3339ebc18
tree96c2a8a4cda9b5d23f5f12a2787111cbb7d6d546
parent6dca2601294f9604b84f46980b84d02c0a36836a
Avoid unexpected conversion overflow in planner for distant date values.

The "date" type supports a wider range of dates than int64 timestamps do.
However, there is pre-int64-timestamp code in the planner that assumes that
all date values can be converted to timestamp with impunity.  Fortunately,
what we really need out of the conversion is always a double (float8)
value; so even when the date is out of timestamp's range it's possible to
produce a sane answer.  All we need is a code path that doesn't try to
force the result into int64.  Per trouble report from David Rericha.

Back-patch to all supported versions.  Although this is surely a corner
case, there's not much point in advertising a date range wider than
timestamp's if we will choke on such values in unexpected places.
src/backend/utils/adt/date.c
src/backend/utils/adt/selfuncs.c
src/include/utils/date.h