nodejs
diff --git a/Collapse file
‎src/string_decoder.cc‎
Copy file name to clipboardExpand all lines: src/string_decoder.cc
+8-1Lines changed: 8 additions & 1 deletion b/Collapse file
‎src/string_decoder.cc‎
Copy file name to clipboardExpand all lines: src/string_decoder.cc
+8-1Lines changed: 8 additions & 1 deletion
diff --git a/Collapse file
‎test/parallel/test-string-decoder.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-string-decoder.js
+7Lines changed: 7 additions & 0 deletions b/Collapse file
‎test/parallel/test-string-decoder.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-string-decoder.js
+7Lines changed: 7 additions & 0 deletions
@@ -3,6 +3,7 @@
 
 #include "env-inl.h"
 #include "node_buffer.h"
+#include "node_errors.h"
 #include "node_external_reference.h"
 #include "string_bytes.h"
 #include "util.h"
@@ -30,11 +31,17 @@ MaybeLocal<String> MakeString(Isolate* isolate,
   Local<Value> error;
   MaybeLocal<Value> ret;
   if (encoding == UTF8) {
-    return String::NewFromUtf8(
+    MaybeLocal<String> utf8_string = String::NewFromUtf8(
         isolate,
         data,
         v8::NewStringType::kNormal,
         length);
+    if (utf8_string.IsEmpty()) {
+      isolate->ThrowException(node::ERR_STRING_TOO_LONG(isolate));
+      return MaybeLocal<String>();
+    } else {
+      return utf8_string;
+    }
   } else {
     ret = StringBytes::Encode(
         isolate,
 
@@ -201,6 +201,13 @@ assert.throws(
   }
 );
 
+assert.throws(
+  () => new StringDecoder().write(Buffer.alloc(0x1fffffe8 + 1).fill('a')),
+  {
+    code: 'ERR_STRING_TOO_LONG',
+  }
+);
+
 // Test verifies that StringDecoder will correctly decode the given input
 // buffer with the given encoding to the expected output. It will attempt all
 // possible ways to write() the input buffer, see writeSequences(). The