harmony-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Elena Semukhina (JIRA)" <j...@apache.org>
Subject [jira] Updated: (HARMONY-1720) Classlib test org.apache.harmony.luni.tests.java.lang.ThreadTest.test_stop_subtest0 fails
Date Mon, 16 Oct 2006 11:18:36 GMT
     [ http://issues.apache.org/jira/browse/HARMONY-1720?page=all ]

Elena Semukhina updated HARMONY-1720:
-------------------------------------

    Attachment: ThreadStopThrowable.java
                H-1720_combined.patch

Alexey,

I attached the new patch which combines both suggestions. I added synchronized block to stop()
as you noticed in 1) above. As for 2), the RI's behavior is asymmetric for stop() and stop(Throwable)
for the threads that are not alive. The attached test ThreadStopThrowable.java demonstrates
this.

Your comments?

> Classlib test org.apache.harmony.luni.tests.java.lang.ThreadTest.test_stop_subtest0 fails
> -----------------------------------------------------------------------------------------
>
>                 Key: HARMONY-1720
>                 URL: http://issues.apache.org/jira/browse/HARMONY-1720
>             Project: Harmony
>          Issue Type: Bug
>          Components: DRLVM
>            Reporter: Elena Semukhina
>            Priority: Critical
>         Attachments: H-1720.patch, H-1720_combined.patch, Harmony-1720_Thread.patch,
ThreadStop1.java, ThreadStopThrowable.java
>
>
> The test fails with the following assertion:
> test_stop_subtest0(org.apache.harmony.luni.tests.java.lang.ThreadTest)junit.fework.AssertionFailedError:
checkAccess called
> The test does not expect that security manager's checkAccess() method is called when
stopping a new thread which is not running yet.
> I think it is a compatibility issue. 
> The spec for Thread.stop() says:
> If there is a security manager installed, its checkAccess method is called with this
as its argument.
> RI does not call this method for a thread which does not run, and this seems reasonable.
> I suggest that drlvm would do the same.
> NOTE: to reproduce the bug you should comment out ThreadTest.test_suspend() which hangs
now. This issue is expected to be fixed with H-1519.

-- 
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

        

Mime
View raw message