src: fix to use replacement character

cola119 · ruyadorno · commit 728e18e0252e · 2022-08-22T19:37:10.000-04:00
PR-URL: #43999 Fixes: #43962 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com> Reviewed-By: Darshan Sen <raisinten@gmail.com> Reviewed-By: LiviaMedeiros <livia@cirno.name> Reviewed-By: Feng Yu <F3n67u@outlook.com>
diff --git a/src/node_i18n.cc b/src/node_i18n.cc
@@ -425,7 +425,11 @@ void ConverterObject::Create(const FunctionCallbackInfo<Value>& args) {
 nullptr, nullptr, nullptr, &status);
 }
 
- new ConverterObject(env, obj, conv, flags);
+ auto converter = new ConverterObject(env, obj, conv, flags);
+ size_t sublen = ucnv_getMinCharSize(conv);
+ std::string sub(sublen, '?');
+ converter->set_subst_chars(sub.c_str());
+
 args.GetReturnValue().Set(obj);
 }
 
diff --git a/test/parallel/test-whatwg-encoding-custom-textdecoder.js b/test/parallel/test-whatwg-encoding-custom-textdecoder.js
@@ -199,3 +199,10 @@ if (common.hasIntl) {
 const str = decoder.decode(chunk);
 assert.strictEqual(str, 'foo\ufffd');
 }
+
+if (common.hasIntl) {
+ const decoder = new TextDecoder('Shift_JIS');
+ const chunk = new Uint8Array([-1]);
+ const str = decoder.decode(chunk);
+ assert.strictEqual(str, '\ufffd');
+}

Original file line number	Diff line number	Diff line change
`@@ -425,7 +425,11 @@ void ConverterObject::Create(const FunctionCallbackInfo<Value>& args) {`
`425`	`425`	`nullptr, nullptr, nullptr, &status);`
`426`	`426`	`}`
`427`	`427`
`428`		`-newConverterObject(env, obj, conv, flags);`
	`428`	`+auto converter = newConverterObject(env, obj, conv, flags);`
	`429`	`+size_t sublen = ucnv_getMinCharSize(conv);`
	`430`	`+ std::string sub(sublen, '?');`
	`431`	`+ converter->set_subst_chars(sub.c_str());`
	`432`	`+`
`429`	`433`	`args.GetReturnValue().Set(obj);`
`430`	`434`	`}`
`431`	`435`
-Original file line number
+Diff line change
 conststr=decoder.decode(chunk);
 assert.strictEqual(str,'foo\ufffd');
+}
++
 +if(common.hasIntl){
 +constdecoder=newTextDecoder('Shift_JIS');
 +constchunk=newUint8Array([-1]);
 +conststr=decoder.decode(chunk);
 +assert.strictEqual(str,'\ufffd');
 +}