7056489: test/com/sun/jndi/ldap/ReadTimeoutTest.java loops or times out

2011-06-18 Thread Alan Bateman
I need a reviewer for a small fix to a test that I caught looping on my system. The test creates a server thread that emulates a non responding LDAP server. Unfortunately the thread goes into a loop reading from the connection once it is closed. I'll bet the original author only tested this

Re: 7056489: test/com/sun/jndi/ldap/ReadTimeoutTest.java loops or times out

2011-06-18 Thread RĂ©mi Forax
On 06/18/2011 09:39 PM, Alan Bateman wrote: I need a reviewer for a small fix to a test that I caught looping on my system. The test creates a server thread that emulates a non responding LDAP server. Unfortunately the thread goes into a loop reading from the connection once it is closed.

Re: 7056489: test/com/sun/jndi/ldap/ReadTimeoutTest.java loops or times out

2011-06-18 Thread Vincent Ryan
Your fix looks good. On 18/06/2011 20:39, Alan Bateman wrote: I need a reviewer for a small fix to a test that I caught looping on my syst.em. The test creates a server thread that emulates a non responding LDAP server. Unfortunately the thread goes into a loop reading from the connection once