http: Do not free the wrong parser on socket close

This appears to fix #4673. That bug is very hard to reproduce, so it's hard to tell for certain, but this approach is more correct anyway. Hat-tip: @dougwilson
nodejs · Feb 25, 2013 · b0e7dbf · b0e7dbf
1 parent f9a0140
commit b0e7dbf
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/lib/http.js b/lib/http.js
@@ -1788,7 +1788,8 @@ function connectionListener(socket) {
   function serverSocketCloseListener() {
     debug('server socket close');
     // mark this parser as reusable
-    freeParser(parser);
+    if (this.parser)
+      freeParser(this.parser);
 
     abortIncoming();
   }