Fix for small leak in TimeZone_md.c

Andrew Hughes gnu.andrew at redhat.com
Wed Aug 19 15:17:05 UTC 2015



----- Original Message -----
> 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
> 
> 

The revised change looks good to me. Feel free to add me as a second reviewer if
you still need one.

Thanks,
-- 
Andrew :)

Senior Free Java Software Engineer
Red Hat, Inc. (http://www.redhat.com)

PGP Key: ed25519/35964222 (hkp://keys.gnupg.net)
Fingerprint = 5132 579D D154 0ED2 3E04  C5A0 CFDA 0F9B 3596 4222

PGP Key: rsa4096/248BDC07 (hkp://keys.gnupg.net)
Fingerprint = EC5A 1F5E C0AD 1D15 8F1F  8F91 3B96 A578 248B DC07



More information about the jdk9-dev mailing list