Code Review 6954525: Testcase failure java/net/Authenticator/B4769350.java

2010-06-23 Thread Chris Hegarty

Hi Michael, Max,

This is a code review request for CR 6954525: Testcase failure 
java/net/Authenticator/B4769350.java.


Essentially, there is a race among requesting threads in the test. The 
threads are making HTTP requests requiring authentication. This is fine, 
and what the test is trying to achieve, but some of the final replies 
from the server/proxy closed the connection after they send the response 
without notifying the HTTP client. If these connections go back into the 
keep alive cache they may cause problems when/if reused by another 
thread. See bug eval section for more details.


The solution is to include the 'Connection: close' header if the 
server/proxy is going to close the connection.


Also, sun.net.www.protocol.http.HttpURLConnection.disconnectWeb should 
check if the connection is to be kept alive before automatically resetting.


Webrev:
  http://cr.openjdk.java.net/~chegar/6954525/webrev.00/webrev/

Thanks,
-Chris.


hg: jdk7/tl/jdk: 6931566: NetworkInterface is not working when interface name is more than 15 characters long

2010-06-23 Thread dmitry . samersoff
Changeset: 887e525597f8
Author:dsamersoff
Date:  2010-06-23 17:25 +0400
URL:   http://hg.openjdk.java.net/jdk7/tl/jdk/rev/887e525597f8

6931566: NetworkInterface is not working when interface name is more than 15 
characters long
Summary: Separate Linux and Solaris code, use lifreq under Solaris
Reviewed-by: chegar

! src/solaris/native/java/net/NetworkInterface.c



hg: jdk7/tl/jdk: 2 new changesets

2010-06-23 Thread alan . bateman
Changeset: eb84b89ef3ff
Author:alanb
Date:  2010-06-23 20:19 +0100
URL:   http://hg.openjdk.java.net/jdk7/tl/jdk/rev/eb84b89ef3ff

6963027: TEST_BUG: channels and buffer tests need to run in samevm mode
Reviewed-by: ohair, sherman, chegar

! test/Makefile
! test/ProblemList.txt
! test/java/nio/BufferPoolMXBean/Basic.java
! test/java/nio/MappedByteBuffer/Basic.java
! test/java/nio/MappedByteBuffer/Force.java
! test/java/nio/MappedByteBuffer/ZeroMap.java
! test/java/nio/channels/AsynchronousChannelGroup/GroupOfOne.java
! test/java/nio/channels/AsynchronousChannelGroup/Identity.java
! test/java/nio/channels/AsynchronousDatagramChannel/Basic.java
! test/java/nio/channels/AsynchronousFileChannel/Basic.java
! test/java/nio/channels/AsynchronousFileChannel/Lock.java
! test/java/nio/channels/AsynchronousFileChannel/LotsOfWrites.java
! test/java/nio/channels/AsynchronousSocketChannel/Basic.java
! test/java/nio/channels/Channels/Basic2.java
! test/java/nio/channels/Channels/Write.java
! test/java/nio/channels/DatagramChannel/AdaptDatagramSocket.java
! test/java/nio/channels/DatagramChannel/EmptyBuffer.java
! test/java/nio/channels/DatagramChannel/ReceiveISA.java
! test/java/nio/channels/DatagramChannel/SelectWhenRefused.java
! test/java/nio/channels/FileChannel/Args.java
! test/java/nio/channels/FileChannel/ClosedChannelTransfer.java
! test/java/nio/channels/FileChannel/ExpandingMap.java
! test/java/nio/channels/FileChannel/Lock.java
! test/java/nio/channels/FileChannel/MapOverEnd.java
! test/java/nio/channels/FileChannel/MapReadOnly.java
! test/java/nio/channels/FileChannel/MapTest.java
! test/java/nio/channels/FileChannel/Mode.java
! test/java/nio/channels/FileChannel/Position.java
! test/java/nio/channels/FileChannel/Pread.java
! test/java/nio/channels/FileChannel/Pwrite.java
! test/java/nio/channels/FileChannel/Read.java
! test/java/nio/channels/FileChannel/ReadFull.java
! test/java/nio/channels/FileChannel/ReadToLimit.java
! test/java/nio/channels/FileChannel/ReleaseOnCloseDeadlock.java
! test/java/nio/channels/FileChannel/ScatteringRead.java
! test/java/nio/channels/FileChannel/Size.java
! test/java/nio/channels/FileChannel/Transfer.java
! test/java/nio/channels/FileChannel/TransferToChannel.java
! test/java/nio/channels/FileChannel/TransferToNonWritable.java
! test/java/nio/channels/FileChannel/Transfers.java
! test/java/nio/channels/FileChannel/TryLock.java
! test/java/nio/channels/FileChannel/Write.java
! test/java/nio/channels/Pipe/NonBlocking.java
! test/java/nio/channels/Pipe/SelectPipe.java
! test/java/nio/channels/SelectionKey/AtomicAttachTest.java
! test/java/nio/channels/Selector/BasicAccept.java
! test/java/nio/channels/Selector/BasicConnect.java
! test/java/nio/channels/Selector/CheckLocking.java
! test/java/nio/channels/Selector/CloseInvalidatesKeys.java
! test/java/nio/channels/Selector/CloseWhenKeyIdle.java
! test/java/nio/channels/Selector/Connect.java
! test/java/nio/channels/Selector/ConnectWrite.java
! test/java/nio/channels/Selector/HelperSlowToDie.java
! test/java/nio/channels/Selector/KeysReady.java
! test/java/nio/channels/Selector/LotsOfChannels.java
! test/java/nio/channels/Selector/RegAfterPreClose.java
! test/java/nio/channels/Selector/SelectAndCancel.java
! test/java/nio/channels/Selector/SelectorLimit.java
! test/java/nio/channels/Selector/SelectorTest.java
! test/java/nio/channels/Selector/WakeupNow.java
! test/java/nio/channels/Selector/WakeupOverflow.java
! test/java/nio/channels/Selector/WakeupSpeed.java
- test/java/nio/channels/ServerSocketChannel/AcceptAddress.java
! test/java/nio/channels/SocketChannel/AdaptSocket.java
! test/java/nio/channels/SocketChannel/Bind.java
! test/java/nio/channels/SocketChannel/Close.java
! test/java/nio/channels/SocketChannel/CloseRegisteredChannel.java
! test/java/nio/channels/SocketChannel/CloseTimeoutChannel.java
! test/java/nio/channels/SocketChannel/IsConnectable.java
! test/java/nio/channels/SocketChannel/LocalAddress.java
! test/java/nio/channels/SocketChannel/SocketInheritance.java
! test/java/nio/channels/SocketChannel/Trivial.java
! test/java/nio/channels/SocketChannel/UnboundSocketTests.java
! test/java/nio/channels/etc/Shadow.java
! 
test/java/nio/channels/spi/SelectorProvider/inheritedChannel/ClosedStreams.java
! test/sun/nio/ch/Basic.java
! test/sun/nio/ch/TempBuffer.java
! test/sun/nio/cs/ReadZero.java
! test/sun/nio/cs/Test4206507.java
! test/sun/nio/cs/TestStringCoding.java

Changeset: 55aa27b8bb98
Author:alanb
Date:  2010-06-23 21:22 +0100
URL:   http://hg.openjdk.java.net/jdk7/tl/jdk/rev/55aa27b8bb98

Merge




hg: jdk7/tl/jdk: 6911258: Project Coin: Add essential API support for Automatic Resource Management (ARM) blocks; ...

2010-06-23 Thread joe . darcy
Changeset: c4d60bcce958
Author:darcy
Date:  2010-06-23 17:03 -0700
URL:   http://hg.openjdk.java.net/jdk7/tl/jdk/rev/c4d60bcce958

6911258: Project Coin: Add essential API support for Automatic Resource 
Management (ARM) blocks
6911261: Project Coin: Retrofit Automatic Resource Management (ARM) support 
onto platform APIs
6962571: Infinite loop in printing out Throwable stack traces with circular 
references
Reviewed-by: darcy, alanb
Contributed-by: j...@google.com

! make/java/java/FILES_java.gmk
! src/share/classes/java/io/Closeable.java
+ src/share/classes/java/lang/AutoCloseable.java
! src/share/classes/java/lang/Throwable.java
! src/share/classes/java/nio/channels/FileLock.java
! src/share/classes/javax/imageio/stream/ImageInputStream.java
+ test/java/lang/Throwable/SuppressedExceptions.java



hg: jdk7/tl/jdk: 3 new changesets

2010-06-23 Thread weijun . wang
Changeset: 706e2d1fc378
Author:weijun
Date:  2010-06-24 14:26 +0800
URL:   http://hg.openjdk.java.net/jdk7/tl/jdk/rev/706e2d1fc378

6958026: Problem with PKCS12 keystore
Reviewed-by: mullan

! src/share/classes/sun/security/pkcs12/PKCS12KeyStore.java
+ test/sun/security/pkcs12/PKCS12SameKeyId.java

Changeset: 1da7dfca3e20
Author:weijun
Date:  2010-06-24 14:26 +0800
URL:   http://hg.openjdk.java.net/jdk7/tl/jdk/rev/1da7dfca3e20

6844907: krb5 etype order should be from strong to weak
Reviewed-by: valeriep

! src/share/classes/sun/security/krb5/Credentials.java
! src/share/classes/sun/security/krb5/internal/crypto/EType.java
! src/share/classes/sun/security/krb5/internal/ktab/KeyTab.java
+ test/sun/security/krb5/etype/ETypeOrder.java
! test/sun/security/krb5/ktab/HighestKvno.java

Changeset: 9c0f542c8b37
Author:weijun
Date:  2010-06-24 14:26 +0800
URL:   http://hg.openjdk.java.net/jdk7/tl/jdk/rev/9c0f542c8b37

6946669: SSL/Krb5 should not call EncryptedData.reset(data, false)
Reviewed-by: xuelei

! src/share/classes/sun/security/krb5/EncryptedData.java
! src/share/classes/sun/security/krb5/KrbApRep.java
! src/share/classes/sun/security/krb5/KrbApReq.java
! src/share/classes/sun/security/krb5/KrbAsRep.java
! src/share/classes/sun/security/krb5/KrbCred.java
! src/share/classes/sun/security/krb5/KrbPriv.java
! src/share/classes/sun/security/krb5/KrbTgsRep.java
! src/share/classes/sun/security/ssl/krb5/KerberosClientKeyExchangeImpl.java
! src/share/classes/sun/security/ssl/krb5/KerberosPreMasterSecret.java
! test/sun/security/krb5/auto/SSL.java