Changeset 620 for trunk/sx/io.c

Timestamp:

05/07/08 01:15:08 (5 months ago)

Author:

markdoliner

Message:

Fix a potential memory leak when the remote host closes a TCP
connection. This may not actually be a problem in practice. If it
was it seems like we would have noticed it in valgrind before now
and fixed it already.

I'm also adding a note about my change from revision 619 to the
ChangeLog?, and changing the documentation in mio.h a little to hopefully
make it more clear.

Files:

• trunk/sx/io.c (modified) (1 diff)

trunk/sx/io.c

@@ -205 +205 @@
 
     /* EOF if we got a 0-byte read from the socket */
-    if(read == 0)
+    if(read == 0) {
         /* they went away */
+        _sx_buffer_free(in);
         _sx_state(s, state_CLOSING);
     
-    else {
+    } else {
         _sx_debug(ZONE, "passed %d read bytes", in->len);