diff options
| author | Michael Paquier <michael@paquier.xyz> | 2025-08-07 11:02:04 +0900 |
|---|---|---|
| committer | Michael Paquier <michael@paquier.xyz> | 2025-08-07 11:02:04 +0900 |
| commit | 2242b26ce472db9ac69dc71008c566ea9cd3a5fd (patch) | |
| tree | 86f8e4abb848d76b86556eaeda25da3b2af2115a /src/backend/optimizer/path/allpaths.c | |
| parent | 9ea3b6f751abd7701f3f32d9df26d66410754c94 (diff) | |
Fix incorrect Datum conversion in timestamptz_trunc_internal()
The code used a PG_RETURN_TIMESTAMPTZ() where the return type is
TimestampTz and not a Datum.
On 64-bit systems, there is no effect since this just ends up casting
64-bit integers back and forth. On 32-bit systems, timestamptz is
pass-by-reference. PG_RETURN_TIMESTAMPTZ() allocates new memory and
returns the address, meaning that the caller could interpret this as a
timestamp value.
The effect is using "date_trunc(..., 'infinity'::timestamptz) will
return random values (instead of the correct return value 'infinity').
Bug introduced in commit d85ce012f99f.
Author: Peter Eisentraut <peter@eisentraut.org>
Reviewed-by: Tom Lane <tgl@sss.pgh.pa.us>
Discussion: https://postgr.es/m/2d320b6f-b4af-4fbc-9eec-5d0fa15d187b@eisentraut.org
Discussion: https://postgr.es/m/4bf60a84-2862-4a53-acd5-8eddf134a60e@eisentraut.org
Backpatch-through: 18
Diffstat (limited to 'src/backend/optimizer/path/allpaths.c')
0 files changed, 0 insertions, 0 deletions