aboutsummaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorTom Lane <tgl@sss.pgh.pa.us>2017-05-07 11:34:31 -0400
committerTom Lane <tgl@sss.pgh.pa.us>2017-05-07 11:34:31 -0400
commit5788a5670e4a58049b8adc82c4fef97a2c3be327 (patch)
tree59cbdf1dd61c5f88c7b6dcf5cbc58cda50dbe0fc /src
parent628462bda908873688ce738a191b470ab769d604 (diff)
downloadpostgresql-5788a5670e4a58049b8adc82c4fef97a2c3be327.tar.gz
postgresql-5788a5670e4a58049b8adc82c4fef97a2c3be327.zip
Restore fullname[] contents before falling through in pg_open_tzfile().
Fix oversight in commit af2c5aa88: if the shortcut open() doesn't work, we need to reset fullname[] to be just the name of the toplevel tzdata directory before we fall through into the pre-existing code. This failed to be exposed in my (tgl's) testing because the fall-through path is actually never taken under normal circumstances. David Rowley, per report from Amit Kapila Discussion: https://postgr.es/m/CAA4eK1LC7CaNhRAQ__C3ht1JVrPzaAXXhEJRnR5L6bfYHiLmWw@mail.gmail.com
Diffstat (limited to 'src')
-rw-r--r--src/timezone/pgtz.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/src/timezone/pgtz.c b/src/timezone/pgtz.c
index 31dbf8993b1..846f8898ffb 100644
--- a/src/timezone/pgtz.c
+++ b/src/timezone/pgtz.c
@@ -105,6 +105,7 @@ pg_open_tzfile(const char *name, char *canonname)
if (result >= 0)
return result;
/* If that didn't work, fall through to do it the hard way */
+ fullname[fullnamelen] = '\0';
}
/*