Fix memory leak in tzparser.c

[Shixin Wang]

Hi hackers,

I noticed a memory leak in the addToArray() function in 
src/backend/utils/misc/tzparser.c.
When the override parameter is true and a duplicate timezone abbreviation is 
found,
the code overwrites midptr->zone without freeing the previously allocated 
memory.

The fix would be:

-               midptr->zone = entry->zone;
+               if (midptr->zone != NULL)
+                   pfree(midptr->zone);
+               midptr->zone = entry->zone;

While the memory is managed by a temp memory context that gets cleaned up
eventually, the coarse-grained management might cause some memory to
accumulate during ParseTzFile() recursive calls when processing @INCLUDE
directives.

I've attached a patch with this change in case anyone thinks it's worth
applying.

Regards,
Shixin Wang

v1-0001-Fix-memory-leak-in-tzparser.patch
Description: v1-0001-Fix-memory-leak-in-tzparser.patch