]> granicus.if.org Git - postgresql/commit
Avoid statically allocating gmtsub()'s timezone workspace.
authorTom Lane <tgl@sss.pgh.pa.us>
Tue, 16 Oct 2018 15:50:19 +0000 (11:50 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Tue, 16 Oct 2018 15:50:19 +0000 (11:50 -0400)
commit5ef71d4bd3226bfd4e733a2fc0b6598df0cc26cf
tree5c54d8af26cf7de4c06dce3b48102d5f2e4e5cd2
parentca361554c24e51eed38c8975780fa7dd98d760ed
Avoid statically allocating gmtsub()'s timezone workspace.

localtime.c's "struct state" is a rather large object, ~23KB.  We were
statically allocating one for gmtsub() to use to represent the GMT
timezone, even though that function is not at all heavily used and is
never reached in most backends.  Let's malloc it on-demand, instead.

This does pose the question of how to handle a malloc failure, but
there's already a well-defined error report convention here, ie
set errno and return NULL.

We have but one caller of pg_gmtime in HEAD, and two in back branches,
neither of which were troubling to check for error.  Make them do so.
The possible errors are sufficiently unlikely (out-of-range timestamp,
and now malloc failure) that I think elog() is adequate.

Back-patch to all supported branches to keep our copies of the IANA
timezone code in sync.  This particular change is in a stanza that
already differs from upstream, so it's a wash for maintenance purposes
--- but only as long as we keep the branches the same.

Discussion: https://postgr.es/m/20181015200754.7y7zfuzsoux2c4ya@alap3.anarazel.de
src/backend/utils/adt/nabstime.c
src/backend/utils/adt/timestamp.c
src/timezone/localtime.c