net: check status code in afterWrite

Fixes memory leak and spin on writing to dead fds. This was tested in production.
nodejs · Dec 12, 2011 · 8295c80 · 8295c80
1 parent e698dd3
commit 8295c80
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/lib/net.js b/lib/net.js
@@ -465,7 +465,11 @@ function afterWrite(status, handle, req, buffer) {
   if (self.destroyed) {
     return;
   }
-  // TODO check status.
+
+  if (status) {
+    self.destroy(errnoException(errno, 'write'));
+    return;
+  }
 
   timers.active(this);