Hi Igor, Thanks for working on fixing this.
Your fix is indeed completely different from ours. Our fix was applied to openjdk6. We will soon be updating our openjdk6, and try to switch to your fix, after verifying that our crash remains fixed. It is a little troubling that we don't have a simple recipe to reproduce this crash. Perhaps one will be forthcoming from us. I recommend backporting this to jdk6, since it is a jdk crash fix. Martin On Wed, Dec 17, 2008 at 11:18, Igor Nekrestyanov <[email protected]> wrote: > Fix for this problem was integrated. > > Note that it is different from suggested one (see > http://hg.openjdk.java.net/jdk7/2d/jdk/rev/11d333de082f) as approach taken > follows what we do for > other rasterizer more closely. > > The idea is to ensure cached state is up to date. Then we do not need > neither global JNIEnv nor global reference to font2d object. > > The caveat here is that i was not able to reproduce problem locally and > therefore fix was not tested properly. > Please let me know if it does not resolve the problem in your environment. > > -igor > > Martin Buchholz wrote: >> >> Ping. >> >> A crash found in testing by users seems pretty high priority. >> >> Martin >> >> On Tue, Oct 21, 2008 at 02:13, Igor Nekrestyanov >> <[email protected]> wrote: >> >>> >>> Thanks! >>> >>> I've filed bug 6761791 and plan to evaluate suggested fix in more >>> details/fix the bug soon. >>> >>> -igor >>> >>> Martin Buchholz wrote: >>> >>>> >>>> Igor, I did some more digging, and I have a stack trace for you >>>> >>>> Native frames: (J=compiled Java code, j=interpreted, Vv=VM code, >>>> C=native >>>> code) >>>> V [libjvm.so+0x298366] >>>> C [libfontmanager.so+0x59e4] >>>> C [libfreetype.so.6+0x73c9] FT_Stream_Close+0x19 >>>> C [libfreetype.so.6+0xa065] FT_Stream_Free+0x25 >>>> C [libfreetype.so.6+0xa6e2] >>>> C [libfreetype.so.6+0xaf78] FT_Done_Face+0x78 >>>> C [libfontmanager.so+0x6964] >>>> Java_sun_font_FreetypeFontScaler_disposeNativeScaler+0x34 >>>> >>>> (sorry for the lack of debug info) >>>> >>>> (These are all running on Linux) >>>> >>>> Hope this helps. >>>> >>>> Martin >>>> >>>> >>> >>> > >
