jnr / jnr-unixsocket

UNIX domain sockets (AF_UNIX) for Java
Apache License 2.0
278 stars 75 forks source link

Fix #60 (calling close on a UnixSocket with a pending read waits for the read to time out on Linux) #68

Closed charleskorn closed 4 years ago

charleskorn commented 5 years ago

See #60 for details.

charleskorn commented 5 years ago

I have tested this on both Linux and OS X, this change may reveal other issues on other OSes.

rhuss commented 4 years ago

Any chance to get this PR merged and maybe a new release ?

rhuss commented 4 years ago

@headius Is there anything I can help to get a release with this fix included? It looks like that this PR fixes are long outstanding (and difficult) bug in d-m-p when accessing the same socket from multiple threads. Unfortunately I'm not deep enough in the weeds to make a qualified review for this PR but I got reports that this will indeed fix that prio1 bug over there in d-m-p land.

Happy new year btw ;-)

headius commented 4 years ago

Sorry about delays...cycling back through JNR stuff now!

headius commented 4 years ago

I'll push out a release today!

rhuss commented 4 years ago

@headius awesome, thanks a ton !