]> granicus.if.org Git - postgresql/commit
Reject out-of-range numeric timezone specifications.
authorTom Lane <tgl@sss.pgh.pa.us>
Tue, 22 Jul 2014 02:41:30 +0000 (22:41 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Tue, 22 Jul 2014 02:41:30 +0000 (22:41 -0400)
commitf54d97c5ef58b436a21373537f85bf5ac8159d8c
tree7c098470c049de7085828f84cf802a31f45c81c4
parente1ea61a30121a97eee192adc0808635fcf7b6f25
Reject out-of-range numeric timezone specifications.

In commit 631dc390f49909a5c8ebd6002cfb2bcee5415a9d, we started to handle
simple numeric timezone offsets via the zic library instead of the old
CTimeZone/HasCTZSet kluge.  However, we overlooked the fact that the zic
code will reject UTC offsets exceeding a week (which seems a bit arbitrary,
but not because it's too tight ...).  This led to possibly setting
session_timezone to NULL, which results in crashes in most timezone-related
operations as of 9.4, and crashes in a small number of places even before
that.  So check for NULL return from pg_tzset_offset() and report an
appropriate error message.  Per bug #11014 from Duncan Gillis.

Back-patch to all supported branches, like the previous patch.
(Unfortunately, as of today that no longer includes 8.4.)
src/backend/commands/variable.c
src/timezone/pgtz.c