Sergey,

It is ok to have not thread safe code in AWT implementation which can
work incorrectly in some cases. It is not ok (IMHO) if the incorrect
work leads to VM crash. It can be problem as a security hole.

--
Best regards,
Ivan

On 11/17/06, Sergey Soldatov (JIRA) <[EMAIL PROTECTED]> wrote:
    [ 
http://issues.apache.org/jira/browse/HARMONY-2100?page=comments#action_12450683 
]

Sergey Soldatov commented on HARMONY-2100:
------------------------------------------

I'm not sure there is any reason to do it, since this functionality is not 
supposed to be run from several threads. AWT itself is not threadsafe and 
documentation recommends to make all AWT calls from event dispatch thread. I 
would keep it as is.

> [classlib] java.awt.color.ICC_ProfileRTest fails
> ------------------------------------------------
>
>                 Key: HARMONY-2100
>                 URL: http://issues.apache.org/jira/browse/HARMONY-2100
>             Project: Harmony
>          Issue Type: Bug
>          Components: Classlib
>         Environment: SUSE 9
> debug DRLVM
>            Reporter: Alexey Varlamov
>         Assigned To: Alexey Varlamov
>            Priority: Minor
>         Attachments: harmony-2100.patch, quick_fix.diff
>
>
> The java.awt.color.ICC_ProfileRTest passes on JITs and fails on interpreter:
> junit.framework.AssertionFailedError: IllegalArgumentExceptione expected
> at java.awt.color.ICC_ProfileRTest.testGetInstance(ICC_ProfileRTest.java:38)
> at java.lang.reflect.VMReflection.invokeMethod(VMReflection.java)
> To reproduce:
> edit $vm/build/lnx_ia32_gcc_debug/deploy/jre/bin/default/harmonyvm.properties, add 
"-Xint" line to it;
> cd $classlib
> ant -Dtest.jre.home=$vm/build/lnx_ia32_gcc_debug/deploy/jre test 
-Dtest.case=java.awt.color.ICC_ProfileRTest -Dbuild.module=awt

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

--
Ivan
Intel Enterprise Solutions Software Division

Reply via email to