cmusatyalab / opendiamond

Interactive search of non-indexed data
http://diamond.cs.cmu.edu/
Eclipse Public License 1.0
19 stars 9 forks source link

Fixing potential memory leak #51

Open john-science opened 2 years ago

john-science commented 2 years ago

The potential problem I am fixing here occurs when socket.shutdown fails and doesn’t proceed to socket.close to clean up the socket and allow garbage collection to release the memory used for it.

The OSError on shutdown can occur when the remote side of the connection closes the connection first.