Skip to content

Fix crash occured by .m3u8 file saved for Playlist - #39333

Merged
sangwoo108 merged 1 commit into
masterfrom
sko/playlist-crash
Aug 24, 2026
Merged

Fix crash occured by .m3u8 file saved for Playlist#39333
sangwoo108 merged 1 commit into
masterfrom
sko/playlist-crash

Conversation

@sangwoo108

@sangwoo108 sangwoo108 commented Aug 24, 2026

Copy link
Copy Markdown
Collaborator

When a saved .m3u8 points some other relative url,
PlaylistDataSource might be requested to serve unknown url.

For example, in a media.m3u8

/amplify_video/2091454598288449536/pl/avc1/480x270/ESjKu9eJEx6jexZe.m3u8

The relative url is resolvd to
"chrome-untrusted://playlist-data/amplyfy_video..." inheriting
origin from the source url.

So instead of crashing the browser, fallback to unknown type.

Resolved brave/brave-browser#58180

When a saved .m3u8 points some other relative url,
PlaylistDataSource might be requested to serve unknown url.

For example, in a media.m3u8
```
/amplify_video/2091454598288449536/pl/avc1/480x270/ESjKu9eJEx6jexZe.m3u8
```

The relative url on the second line is resolvd to
"chrome-untrusted://playlist-data/amplyfy_video..." inheriting
origin from the source url.

So instead of crashing the browser, fallback to unknown type.
@sangwoo108 sangwoo108 changed the title Playlist crash Fix crash occured by .m3u8 file saved for Playlist Aug 24, 2026
@sangwoo108
sangwoo108 requested review from simonhong and a lite review from Copilot and removed request for Copilot August 24, 2026 05:31

@simonhong simonhong left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

++

@sangwoo108
sangwoo108 merged commit e82b6ee into master Aug 24, 2026
21 checks passed
@sangwoo108
sangwoo108 deleted the sko/playlist-crash branch August 24, 2026 07:54
@brave-builds brave-builds added this to the 1.96.x - Nightly milestone Aug 24, 2026
@brave-builds

Copy link
Copy Markdown
Collaborator

Released in v1.96.10

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Crash Report

3 participants