On 01/03/2013 11:45, Alexey Utkin wrote:

That is possible, but includes pre-requirements for installed MS tools. That is not a good idea.
I agree.


New version of the fix was prepared.
Bug description:
    http://bugs.sun.com/view_bug.do?bug_id=7190897
    https://jbs.oracle.com/bugs/browse/JDK-7190897
The suggested fix:
http://cr.openjdk.java.net/~uta/openjdk-webrevs/JDK-7190897/webrev.01/
This looks much better, thank you.

A minor point is that in checkAccessMask then you could call OpenThreadToken before the try/finally. That way you wouldn't need to check hToken. Otherwise I think this is good to go and it's nice to finally fix this issue.

-Alan




Reply via email to