No longer OOM on invalid string length.

src/json-parser.h

Index: src/json-parser.h
diff --git a/src/json-parser.h b/src/json-parser.h
index 097358932f18df46bb19f85521aefb18eeadc552..924d0922874672fde8c932ab1df0db51f019800f 100644
--- a/src/json-parser.h
+++ b/src/json-parser.h
@@ -606,6 +606,7 @@ Handle<String> JsonParser<seq_ascii>::SlowScanJsonString(
   int length = Min(max_length, Max(kInitialSpecialStringLength, 2 * count));
   Handle<StringType> seq_string =
       NewRawString<StringType>(factory(), length, pretenure_);
+  ASSERT(!seq_string.is_null());
   // Copy prefix into seq_str.
   SinkChar* dest = seq_string->GetChars();
   String::WriteToFlat(*prefix, dest, start, end);
@@ -793,6 +794,8 @@ Handle<String> JsonParser<seq_ascii>::ScanJsonString() {
   } while (c0_ != '"');
   int length = position_ - beg_pos;
   Handle<String> result = factory()->NewRawOneByteString(length, pretenure_);
+  ASSERT(!result.is_null());
+  RETURN_IF_EMPTY_HANDLE_VALUE(isolate(), result, Handle<String>());

[Igor Sheludko, 2014/03/24 13:43:16]

Do we really need both ASSERT and RETURN_IF_EMPTY_HANDLE_VALUE here?

   uint8_t* dest = SeqOneByteString::cast(*result)->GetChars();
   String::WriteToFlat(*source_, dest, beg_pos, position_);