From 267c09abf8e0c443bac7f1a4ced9f78b03d312cf Mon Sep 17 00:00:00 2001 From: Vanilagy <1696106+Vanilagy@users.noreply.github.com> Date: Tue, 13 Jan 2026 17:34:33 +0100 Subject: [PATCH] Return null earlier on --- src/ogg/ogg-demuxer.ts | 8 ++++---- .../{ogg-vorbis-windowed.test.ts => ogg-demuxer.test.ts} | 0 2 files changed, 4 insertions(+), 4 deletions(-) rename test/browser/{ogg-vorbis-windowed.test.ts => ogg-demuxer.test.ts} (100%) diff --git a/src/ogg/ogg-demuxer.ts b/src/ogg/ogg-demuxer.ts index e509933..0b4569f 100644 --- a/src/ogg/ogg-demuxer.ts +++ b/src/ogg/ogg-demuxer.ts @@ -322,6 +322,10 @@ export class OggDemuxer extends Demuxer { } const totalPacketSize = chunks.reduce((sum, chunk) => sum + chunk.length, 0); + if (totalPacketSize === 0) { + return null; // Invalid packet, treat it as end of stream + } + const packetData = new Uint8Array(totalPacketSize); let offset = 0; @@ -499,10 +503,6 @@ class OggAudioTrackBacking implements InputAudioTrackBacking { return null; } - if (packet.data.byteLength === 0) { - return null; - } - const { durationInSamples, vorbisBlockSize } = extractSampleMetadata( packet.data, this.bitstream.codecInfo, diff --git a/test/browser/ogg-vorbis-windowed.test.ts b/test/browser/ogg-demuxer.test.ts similarity index 100% rename from test/browser/ogg-vorbis-windowed.test.ts rename to test/browser/ogg-demuxer.test.ts