http-parser: Error on EOF errors

nodejs · Feb 5, 2011 · c783aef · c783aef
1 parent e42c74e
commit c783aef
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/src/node_http_parser.cc b/src/node_http_parser.cc
@@ -308,10 +308,17 @@ class Parser : public ObjectWrap {
     assert(!current_buffer);
     parser->got_exception_ = false;
 
-    http_parser_execute(&(parser->parser_), &settings, NULL, 0);
+    int rv = http_parser_execute(&(parser->parser_), &settings, NULL, 0);
 
     if (parser->got_exception_) return Local<Value>();
 
+    if (rv != 0) {
+      Local<Value> e = Exception::Error(String::NewSymbol("Parse Error"));
+      Local<Object> obj = e->ToObject();
+      obj->Set(String::NewSymbol("bytesParsed"), Integer::New(0));
+      return scope.Close(e);
+    }
+
     return Undefined();
   }