Reject out-of-range numeric timezone specifications.
authorTom Lane <tgl@sss.pgh.pa.us>
Tue, 22 Jul 2014 02:41:27 +0000 (22:41 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Tue, 22 Jul 2014 02:41:27 +0000 (22:41 -0400)
commit7672bbca0e7b8f2cbdf8a984e13a891d919fde7b
tree9145ff482eb90bbc10ea27cc5e688c9d6cb71d99
parentf7ba173cb3548ddccaab68fcaeae3dd5efdcfbf1
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