buffer: allow invalid encoding in from

Looks like a bug to me but the change should probably done in a semver majpr. PR-URL: #54533 Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Jake Yuesong Li <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]>
nodejs · Oct 2, 2024 · 84bf93b · 84bf93b
1 parent 7f8edce
commit 84bf93b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/lib/buffer.js b/lib/buffer.js
@@ -474,7 +474,7 @@ function createFromString(string, ops, length = ops.byteLength(string)) {
 
 function fromString(string, encoding) {
  let ops;
- if (!encoding || encoding === 'utf8') {
+ if (!encoding || encoding === 'utf8' || typeof encoding !== 'string') {
  ops = encodingOps.utf8;
  } else {
  ops = getEncodingOps(encoding);

diff --git a/test/parallel/test-buffer-from.js b/test/parallel/test-buffer-from.js
@@ -139,3 +139,6 @@ throws(() => {
  code: 'ERR_OUT_OF_RANGE',
  })
 );
+
+// Invalid encoding is allowed
+Buffer.from('asd', 1);