Skip to content
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
chore(mp3info): apply upstream patch for parsing on short frames
Signed-off-by: grnd-alt <[email protected]>
Signed-off-by: Daniel Kesselberg <[email protected]>
  • Loading branch information
grnd-alt authored and backportbot[bot] committed Nov 20, 2025
commit db76f7b4b3e43657c1f4c10a3cd6c29ea59eed5c
26 changes: 26 additions & 0 deletions .patches/mp3info-break-frame-parsing.patch
Original file line number Diff line number Diff line change
@@ -0,0 +1,26 @@
From 186b99ac4a57d091e9414c0944524a9e098835f3 Mon Sep 17 00:00:00 2001
From: grnd-alt <[email protected]>
Date: Mon, 13 Oct 2025 12:18:37 +0200
Subject: [PATCH] fix: break frame parsing on short frame

Signed-off-by: grnd-alt <[email protected]>
---
src/Mp3Info.php | 5 +++++
1 file changed, 5 insertions(+)

diff --git a/src/Mp3Info.php b/src/Mp3Info.php
index ccf97f4..24781d7 100644
--- a/src/Mp3Info.php
+++ b/src/Mp3Info.php
@@ -584,6 +584,11 @@ protected function parseId3v23Body($fp, $lastByte) {
$raw = fread($fp, 10);
$frame_id = substr($raw, 0, 4);

+ if (strlen($raw) < 10) {
+ fseek($fp, $lastByte);
+ break;
+ }
+
if ($frame_id == str_repeat(chr(0), 4)) {
fseek($fp, $lastByte);
break;
5 changes: 5 additions & 0 deletions wapmorgan/mp3info/src/Mp3Info.php
Original file line number Diff line number Diff line change
Expand Up @@ -585,6 +585,11 @@ protected function parseId3v23Body($fp, $lastByte) {
$raw = fread($fp, 10);
$frame_id = substr($raw, 0, 4);

if (strlen($raw) < 10) {
fseek($fp, $lastByte);
break;
}

if ($frame_id == str_repeat(chr(0), 4)) {
fseek($fp, $lastByte);
break;
Expand Down