Fix for small leak in TimeZone_md.c

[Langer, Christoph]

Hi Thomas,

thanks for your review.

Axel already helped me to create a bug report: https://bugs.openjdk.java.net/browse/JDK-8133830

I had also updated my webrev, as I would like to take the chance to do some minor refactoring of the AIX porting code for better readability: http://cr.openjdk.java.net/~asiebenborn/8133830/webrev/

Can you review the updated change as well?

Thanks
Christoph

From: Thomas Stüfe [mailto:thomas.stuefe at gmail.com]
Sent: Mittwoch, 19. August 2015 13:12
To: Langer, Christoph
Cc: jdk9-dev at openjdk.java.net
Subject: Re: Fix for small leak in TimeZone_md.c

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<mailto: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