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

Lindenmaier, Goetz goetz.lindenmaier at sap.com
Tue Aug 25 13:54:28 UTC 2015


Hi Christoph,

the change looks good, Reviewed.

In case this code gets reengineered, you could add a comment that
memory of getenv must not be freed.

Thanks for fixing this,
  Goetz.


-----Original Message-----
From: Langer, Christoph 
Sent: Donnerstag, 20. August 2015 12:12
To: 'jdk9-dev at openjdk.java.net' <jdk9-dev at openjdk.java.net>
Subject: RE: RFR(S): 8133830: Fix for small leak in TimeZone_md.c: 

Hi all,

as per the discussion in thread "Fix for small leak in TimeZone_md.c", I would like to get the following tiny change reviewed:

Bug: https://bugs.openjdk.java.net/browse/JDK-8133830
Webrev: http://cr.openjdk.java.net/~asiebenborn/christoph/webrev/

As said before, I also need a sponsor since I can't commit yet.

Thanks
Christoph


More information about the jdk9-dev mailing list