RFR(S): 8133830: Fix for small leak in TimeZone_md.c:

[Langer, Christoph]

Hi,

we've created a bug report for this issue now: https://bugs.openjdk.java.net/browse/JDK-8133830

I also updated the 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/

Please review this change.  I also need a sponsor.

Thanks
Christoph

-----Original Message-----
From: jdk9-dev [mailto:jdk9-dev-bounces at openjdk.java.net] On Behalf Of Langer, Christoph
Sent: Dienstag, 18. August 2015 11:52
To: jdk9-dev at openjdk.java.net
Subject: Fix for small leak in TimeZone_md.c

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