Fix for small leak in TimeZone_md.c

Dmitry Samersoff dmitry.samersoff at oracle.com
Wed Aug 19 16:53:59 UTC 2015


Lander,

My few cents ...

1. After you refactoring freetz is never actually used. It might be
better to just get rid of this variable.

2. Also you can remove javatz as well and further refactor the code to

    return (tz == NULL) ? NULL : strdup(tz);

-Dmitry

On 2015-08-19 16:14, Langer, Christoph wrote:
> 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
>


-- 
Dmitry Samersoff 
Oracle Java development team, Saint Petersburg, Russia
* I would love to change the world, but they won't give me the sources. 



More information about the jdk9-dev mailing list