Re: [OpenJDK 2D-Dev] [PATCH] Memory leak in LCMS.c

2008-11-07 Thread Mark Wielaard
Hi Roman, On Thu, 2008-11-06 at 21:36 +0100, Roman Kennke wrote: > I found a memory leak in LCMS.c, where one ReleaseByteArrayElements() > call is missing in loadProfile(). I think the IcedTea people discovered > this independently and already included that in IcedTea and I believe > also in OpenJ

[OpenJDK 2D-Dev] [PATCH] Memory leak in LCMS.c

2008-11-06 Thread Roman Kennke
I found a memory leak in LCMS.c, where one ReleaseByteArrayElements() call is missing in loadProfile(). I think the IcedTea people discovered this independently and already included that in IcedTea and I believe also in OpenJDK6, but it doesn't seem to be in JDK7 yet. /Roman -- Dipl.-Inform. (FH

Re: [OpenJDK 2D-Dev] [PATCH] memory leak in LCMS.c

2007-11-22 Thread Alexey Ushakov
Yes, I'll look into it. Best Regards, Alexey Phil Race wrote: This should be 2d-dev. Alexey, can you take care of this. thx, -phil. Thomas Fitzsimmons wrote: Hi, Java_sun_java2d_cmm_lcms_LCMS_loadProfile gets its data parameter's byte array elements but does not release them. A fix is a

Re: [OpenJDK 2D-Dev] [PATCH] memory leak in LCMS.c

2007-11-22 Thread Phil Race
This should be 2d-dev. Alexey, can you take care of this. thx, -phil. Thomas Fitzsimmons wrote: Hi, Java_sun_java2d_cmm_lcms_LCMS_loadProfile gets its data parameter's byte array elements but does not release them. A fix is attached. Tom --