mpv-player / mpv

🎥 Command line video player
https://mpv.io
Other
26.65k stars 2.83k forks source link

demux_playlist: don't add base path to self-expanding protocols #14443

Closed kasper93 closed 2 days ago

kasper93 commented 2 days ago

Adding base path make sense only if it is real directory or url location. In case of protocols like memory adding base path to playlist entry in facts adds whole playlist to that entry.

For example mpv $'memory://#EXTM3U\na/b produces infinite loop, expanding playlist.

Found by OSS-Fuzz.

github-actions[bot] commented 2 days ago

Download the artifacts for this pull request:

Windows * [mpv-i686-w64-mingw32](https://nightly.link/mpv-player/mpv/actions/artifacts/1642260085.zip) * [mpv-x86_64-w64-mingw32](https://nightly.link/mpv-player/mpv/actions/artifacts/1642265833.zip) * [mpv-x86_64-windows-msvc](https://nightly.link/mpv-player/mpv/actions/artifacts/1642283385.zip)
macOS * [mpv-macos-12-intel](https://nightly.link/mpv-player/mpv/actions/artifacts/1642259011.zip) * [mpv-macos-13-intel](https://nightly.link/mpv-player/mpv/actions/artifacts/1642268087.zip) * [mpv-macos-14-arm](https://nightly.link/mpv-player/mpv/actions/artifacts/1642259591.zip)