harmony-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Geir Magnusson Jr <g...@pobox.com>
Subject Re: [jira] Closed: (HARMONY-927) DRLVM's DestroyVM needs to be more robust
Date Thu, 20 Jul 2006 12:15:03 GMT
patch welcome ;)

geir

Pavel Pervov wrote:
> Hm... I commented on dev-list on this issue already in using classlib
> launcher in DRLVM.
> 
> JNI specification explicitly states that:
> ---------------------------------------------------------------------
> After an exception has been raised, the native code must first clear the
> exception before making other JNI calls. When there is a pending exception,
> the only JNI functions that are safe to call are ExceptionOccurred(),
> ExceptionDescribe(), and ExceptionClear().
> ---------------------------------------------------------------------
> See [1] at the end of Exception Handling section (and the end of the page).
> 
> So, it's nice that VM won't crash if entered JNI function in exception
> state, but it is obviously resposibility of native code (launcher in our
> case) to check pending exceptions and process them as necessary _before_
> calling into JNI.
> 
> Regards,
>    Pavel Pervov.
> 
> [1] http://java.sun.com/j2se/1.5.0/docs/guide/jni/spec/design.html#wp770
> 
> On 7/20/06, Geir Magnusson Jr (JIRA) <jira@apache.org> wrote:
>>
>>     [ http://issues.apache.org/jira/browse/HARMONY-927?page=all ]
>>
>> Geir Magnusson Jr closed HARMONY-927.
>> -------------------------------------
>>
>>
>> > DRLVM's DestroyVM needs to be more robust
>> > -----------------------------------------
>> >
>> >                 Key: HARMONY-927
>> >                 URL: http://issues.apache.org/jira/browse/HARMONY-927
>> >             Project: Harmony
>> >          Issue Type: Bug
>> >          Components: DRLVM
>> >            Reporter: Andrey Chernyshev
>> >         Assigned To: Geir Magnusson Jr
>> >         Attachments: destroy_vm.patch
>> >
>> >
>> > drlvm should handle the fact that an exception already exists on
>> entering DestroyJavaVM, and clear it before trying to resolve the VMStart
>> class.
>> > Here is a small patch that adds a simple check into the shutdown
>> method.
>>
>> -- 
>> This message is automatically generated by JIRA.
>> -
>> If you think it was sent incorrectly contact one of the administrators:
>> http://issues.apache.org/jira/secure/Administrators.jspa
>> -
>> For more information on JIRA, see: http://www.atlassian.com/software/jira
>>
>>
>>
> 

---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
For additional commands, e-mail: harmony-dev-help@incubator.apache.org


Mime
View raw message