Raise the correct exception in fast_serialize_string

* Related to #344

Author: eregon

lib/json/pure/generator.rb

@@ -339,6 +339,7 @@ def generate(obj)
           private def fast_serialize_string(string, buf) # :nodoc:
             buf << '"'
             string = string.encode(::Encoding::UTF_8) unless string.encoding == ::Encoding::UTF_8
+            raise GeneratorError, "source sequence is illegal/malformed utf-8" unless string.valid_encoding?
 
             if /["\\\x0-\x1f]/n.match?(string)
               buf << string.gsub(/["\\\x0-\x1f]/n, MAP)

test/json/json_generator_test.rb

@@ -443,6 +443,23 @@ def test_invalid_encoding_string
       "\x82\xAC\xEF".to_json
     end
     assert_includes error.message, "source sequence is illegal/malformed utf-8"
+
+    error = assert_raise(JSON::GeneratorError) do
+      JSON.dump("\x82\xAC\xEF")
+    end
+    assert_includes error.message, "source sequence is illegal/malformed utf-8"
+
+    # These pass on the pure-Ruby generator but not with the native extension
+    # https://github.com/ruby/json/issues/634
+    if defined?(JSON::Pure)
+      assert_raise(Encoding::UndefinedConversionError) do
+        "\x82\xAC\xEF".b.to_json
+      end
+
+      assert_raise(Encoding::UndefinedConversionError) do
+        JSON.dump("\x82\xAC\xEF".b)
+      end
+    end
   end
 
   if defined?(JSON::Ext::Generator) and RUBY_PLATFORM != "java"