Replace binary with latin1 for ISO/IEC 8859-1 text decoding

Borewit · Jun 29, 2024 · b4c8502 · b4c8502
1 parent a49df48
commit b4c8502
Show file tree

Hide file tree

Showing 5 changed files with 5 additions and 5 deletions.
diff --git a/lib/asf/AsfUtil.ts b/lib/asf/AsfUtil.ts
@@ -4,7 +4,7 @@ import { Buffer } from 'node:buffer';
 import * as util from '../common/Util.js';
 import { DataType } from './AsfObject.js';
 
-export type AttributeParser = (buf: Buffer) => boolean | string | number | bigint | Buffer;
+export type AttributeParser = (buf: Uint8Array) => boolean | string | number | bigint | Uint8Array;
 
 export class AsfUtil {
 

diff --git a/lib/id3v1/ID3v1Parser.ts b/lib/id3v1/ID3v1Parser.ts
@@ -157,7 +157,7 @@ export async function hasID3v1Header(reader: IRandomReader): Promise<boolean> {
   if (reader.fileSize >= 128) {
     const tag = Buffer.alloc(3);
     await reader.randomRead(tag, 0, tag.length, reader.fileSize - 128);
-    return tag.toString('binary') === 'TAG';
+    return tag.toString('latin1') === 'TAG';
   }
   return false;
 }
diff --git a/lib/lyrics3/Lyrics3.ts b/lib/lyrics3/Lyrics3.ts
@@ -6,7 +6,7 @@ export async function getLyricsHeaderLength(reader: IRandomReader): Promise<numb
   if (reader.fileSize >= 143) {
     const buf = Buffer.alloc(15);
     await reader.randomRead(buf, 0, buf.length, reader.fileSize - 143);
-    const txt = buf.toString('binary');
+    const txt = buf.toString('latin1');
     const tag = txt.substr(6);
     if (tag === endTag2) {
       return parseInt(txt.substr(0, 6), 10) + 15;

diff --git a/lib/riff/RiffChunk.ts b/lib/riff/RiffChunk.ts
@@ -13,7 +13,7 @@ export const Header: IGetToken<IChunkHeader> = {
   get: (buf: Buffer, off): IChunkHeader => {
     return {
       // Group-ID
-      chunkID: buf.toString('binary', off, off + 4),
+      chunkID: buf.toString('latin1', off, off + 4),
       // Size
       chunkSize: Token.UINT32_LE.get(buf, 4)
     };

diff --git a/test/test-file-asf.ts b/test/test-file-asf.ts
@@ -68,7 +68,7 @@ describe('Parse ASF', () => {
     ];
 
     tests.forEach(test => {
-      const buf = Buffer.from(test.raw, 'binary');
+      const buf = new TextEncoder().encode(test.raw);
       assert.strictEqual(Number(AsfUtil.getParserForAttr(DataType.QWord)(buf)), test.expected, test.description);
     });