update after addaleax' review

doc/api/http.md

@@ -765,10 +765,10 @@ object, so any HTTP response sent, including response headers and payload,
 *must* be written directly to the `socket` object. Care must be taken to
 ensure the response is a properly formatted HTTP response message.
 
-> `err` is an instance of `Error` with two extra columns:
->
-> + `bytesParsed`: the bytes count of request packet that Node.js may parse correctly;
-> + `rawPacket`: the raw packet of current request.
+`err` is an instance of `Error` with two extra columns:
+
++ `bytesParsed`: the bytes count of request packet that Node.js may have parsed correctly;
++ `rawPacket`: the raw packet of current request.
 
 ### Event: 'close'
 <!-- YAML

lib/_http_server.js

@@ -475,8 +475,11 @@ function socketOnError(e) {
 function onParserExecuteCommon(server, socket, parser, state, ret, d) {
   resetSocketTimeout(server, socket, state);
 
+  if (!d)
+    d = parser.getCurrentBuffer();
+
   if (ret instanceof Error) {
-    ret.rawPacket = d || parser.getCurrentBuffer();
+    ret.rawPacket = d;
     debug('parse error', ret);
     socketOnError.call(socket, ret);
   } else if (parser.incoming && parser.incoming.upgrade) {
@@ -485,9 +488,6 @@ function onParserExecuteCommon(server, socket, parser, state, ret, d) {
     var req = parser.incoming;
     debug('SERVER upgrade or connect', req.method);
 
-    if (!d)
-      d = parser.getCurrentBuffer();
-
     socket.removeListener('data', state.onData);
     socket.removeListener('end', state.onEnd);
     socket.removeListener('close', state.onClose);