Hi All,
Please review the fix for JDK-8031661
http://cr.openjdk.java.net/~tyan/amandaj/JDK-8031661/webrev.01/
Description:
Root Cause:
In line 195 and 196, the two actions can be ordered by a happens-before
relationship, which will cause synchronization fail.
t1cond1latch.countDown();
t1cond2
n the updated test for a while and
I can't see any problem with it.
Thanks,
Michael
On 10/02/14 08:56, Amanda Jiang wrote:
Hi All,
Please review the fix for JDK-8031661
http://cr.openjdk.java.net/~tyan/amandaj/JDK-8031661/webrev.01/
<http://cr.openjdk.java.net/%7Etyan/amanda
Hi Michael,
Could you sponsor this for me? Thanks a lot.
Amanda
On 2/10/14 4:10 PM, Amanda Jiang wrote:
> Hi Michael,
>
> Could you sponsor this for me?
>
> Thanks,
> Amanda
>
> *发件人**:*Michael McMahon
> *发送时间**:*Monday, February 10, 2014 6:13 PM
> *收件人**:*net-de
Hi All ,
Please check and review following changes:
http://cr.openjdk.java.net/~ewang/amanda/JDK-8038276/webrev.00/
Description of issue:
Root Cause:
Test java/net/NetworkInterface/Test.java failed because virtual
interface "Teredo Tunneling Pseudo-Interface" on windows returned random
address
Please sponsor this change . Thank you, Chris.
Amanda
On 3/28/14 6:10 AM, Chris Hegarty wrote:
Looks ok to me Amanda. I can sponsor this change, if needed.
-Chris.
On 28/03/14 05:07, Amanda Jiang wrote:
Hi All ,
Please check and review following changes:
http://cr.openjdk.java.net/~ewang
Hi All,
Could you please review 1 new test to be added for SocketPermission.
New test is added to check socket permissions, for instance -
- java.net.SocketPermission with "connect", "resolve", "accept", "listen"
with Socket, DatagramSocket, MulticastSocket etc.
JDK Issue: https://bugs.openjdk.j
Hi Michael, Chris and others
We’re working on moving our internal security tests to open idk
repo(Include refactoring effort). Could you please help to review these
tests?
Thanks,
Amanda
On 7/28/14 5:25 PM, Amanda Jiang wrote:
Hi All,
Could you please review 1 new test to be added for
reate the socket permission instances dynamically accordingly? Or
would this make the test very difficult to debug?
-Chris.
On 25/08/14 17:48, Amanda Jiang wrote:
Hi Michael, Chris and others
We’re working on moving our internal security tests to open idk
repo(Include refactoring effort). Coul
Hi,
May I request your review for following changeset?
There are 33 networking tests that have dependency on internal
APIsun.net.www.MessageHeader and sun.net.www.HeaderParser
This fix is to remove the internal API dependency from those tests.
bug: https://bugs.openjdk.java.net/browse/JDK-80617
Hi,
May I request your review for following changeset?
There is one networking tests that has dependency on internal API
sun.net.www.ParseUtil.
This fix is to remove the internal API dependency from that test.
bug:https://bugs.openjdk.java.net/browse/JDK-8065420
webrev: http://cr.openjdk.java.n
I have updated the fixes to reduce code duplication, please check the
new webrev below:
http://cr.openjdk.java.net/~tyan/amandaj/8061729/webrev/
Thanks,
Amanda
On 12/5/14 1:58 PM, Amanda Jiang wrote:
Hi,
May I request your review for following changeset?
There are 33 networking tests that
Thank you Chris, could you please sponsor this?
Thanks,
Amanda
On 12/18/14 4:18 AM, Chris Hegarty wrote:
This looks ok to me Amanda.
-Chris.
On 17 Dec 2014, at 23:23, Amanda Jiang wrote:
Hi,
May I request your review for following changeset?
There is one networking tests that has
12 matches
Mail list logo