Looks fine to me.

Thanks,
Xuelei

On 11/14/2018 2:21 AM, Sibabrata Sahoo wrote:
Hi Xuelei,

Please review a minor fix for,

JBS: https://bugs.openjdk.java.net/browse/JDK-8211787

Webrev: http://cr.openjdk.java.net/~ssahoo/8211787/webrev.00/

The original intention to “setSoTimeout()” is on serverSocket where the server will fail with timeout if it exceed certain amount of time before accepting a client connection. But by mistake it was set on wrong socket object which is created from accept() and causing a timeout if the InputStream exceed certain amount of time. I have corrected to setSoTimeout() to be set through serverSocket only.

Thanks,

Siba

Reply via email to