[OpenJDK 2D-Dev] [8] request for review: 8024511: Crash during color profile destruction

2013-09-10 Thread Andrew Brygin
Hello, could you please review a fix for CR 8024511? Bug: http://bugs.sun.com/view_bug.do?bug_id=8024511 Webrev: http://cr.openjdk.java.net/~bae/8024511/webrev.00/ Starting fix for 7043064, we are using disposer to destroy a native part of a color profile. However an incorrect disposal method i

Re: [OpenJDK 2D-Dev] [8] request for review: 8024511: Crash during color profile destruction

2013-09-10 Thread Vadim Pakhnushev
Looks good. On 10.09.2013 14:16, Andrew Brygin wrote: Hello, could you please review a fix for CR 8024511? Bug: http://bugs.sun.com/view_bug.do?bug_id=8024511 Webrev: http://cr.openjdk.java.net/~bae/8024511/webrev.00/ Starting fix for 7043064, we are using disposer to destroy a native part of

Re: [OpenJDK 2D-Dev] [8] request for review: 8024511: Crash during color profile destruction

2013-09-10 Thread Phil Race
Oops. fix looks OK. -phil. On 9/10/2013 3:16 AM, Andrew Brygin wrote: Hello, could you please review a fix for CR 8024511? Bug: http://bugs.sun.com/view_bug.do?bug_id=8024511 Webrev: http://cr.openjdk.java.net/~bae/8024511/webrev.00/ Starting fix for 7043064, we are using disposer to destro

[OpenJDK 2D-Dev] hg: jdk8/2d/jdk: 8024511: Crash during color profile destruction

2013-09-10 Thread andrew . brygin
Changeset: 08296c2d4c68 Author:bae Date: 2013-09-10 21:54 +0400 URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/08296c2d4c68 8024511: Crash during color profile destruction Reviewed-by: vadim, prr ! src/share/native/sun/java2d/cmm/lcms/LCMS.c + test/sun/java2d/cmm/ProfileOp/Disposa