Fix for small leak in TimeZone_md.c

[Thomas Stüfe]

Hi Christoph,

looks fine. I opened a bug for you:
https://bugs.openjdk.java.net/browse/JDK-8133933

But this needs another reviewer and a sponsor.

Kind Regards, Thomas



On Tue, Aug 18, 2015 at 11:52 AM, Langer, Christoph <
christoph.langer at sap.com> wrote:

> Hi all,
>
> I think there is a small memory leak in TimeZone_md.c in the case of
> solaris "localtime".
>
> When getPlatformTimeZoneID() is called and its result is "localtime", the
> returned string buffer is not freed as the pointer "freetz" is overwritten
> after the call to getSolarisDefaultZoneID().
>
> Please have a look at my webrev:
> http://cr.openjdk.java.net/~asiebenborn/christoph/webrev/ for a potential
> fix.
>
> A bug was not yet created but should be done after you agree that this is
> an issue.
>
> Thanks in advance for comments.
>
> Best regards
> Christoph
>
>