[OpenJDK 2D-Dev] JNI crashes in FontManager code

[Martin Buchholz]

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
<Igor.Nekrestyanov at sun.com> 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
>> <Igor.Nekrestyanov at sun.com> 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
>>>>
>>>>
>>>
>>>
>
>