8000 Crash when trying to play a stream · Issue #19 · arcnmx/MPD · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Crash when trying to play a stream #19

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
arcnmx opened this issue Oct 19, 2020 · 2 comments
Open

Crash when trying to play a stream #19

arcnmx opened this issue Oct 19, 2020 · 2 comments

Comments

@arcnmx
Copy link
Owner
arcnmx commented Oct 19, 2020

mpc add ytdl://twitch.tv/monstercat and boom:

mpd[7460]: mpd: ../src/input/plugins/../InputStream.hxx:225: InputStream::offset_type InputStream::GetSize() const: Assertion `KnownSize()' failed.
systemd[2437]: mpd.service: Main process exited, code=killed, status=6/ABRT
systemd[2437]: mpd.service: Failed with result 'signal'.
systemd-coredump[7858]: Process 7460 (.mpd-wrapped) of user 1000 dumped core.

This used to work... Adding the youtube-dl -g url directly works so obviously I did something very wrong, maybe with the tag scanner.

@arcnmx
Copy link
Owner Author
arcnmx commented Oct 19, 2020

So the crash is easily fixed by checking for KnownSize(), and presumably would also be fixed by moving to ProxyInputStream. However, fixing it reveals a new problem: I believe the ffmpeg decoder (FfmpegOpenInput) relies on inputstream.GetURI() in some scenarios to actually access the resource. In this case to re-read the HLS/M3U playlist as it updates live. I'm not sure whether just making GetURI virtual or having the decoder request the URL in some other way is the proper fix here...

arcnmx added a commit that referenced this issue Oct 20, 2020
See #19 for details. Also note the lack of synchronization for now.
arcnmx added a commit that referenced this issue Oct 20, 2020
See #19 for details. Also note the lack of synchronization for now.
arcnmx added a commit that referenced this issue Nov 6, 2020
See #19 for details. Also note the lack of synchronization for now.
arcnmx added a commit that referenced this issue Nov 6, 2020
See #19 for details. Also note the lack of synchronization for now.
arcnmx added a commit that referenced this issue Nov 6, 2020
See #19 for details. Also note the lack of synchronization for now.
arcnmx added a commit that referenced this issue Nov 6, 2020
See #19 for details. Also note the lack of synchronization for now.
arcnmx added a commit that referenced this issue Nov 6, 2020
See #19 for details. Also note the lack of synchronization for now.
arcnmx added a commit that referenced this issue Dec 9, 2020
See #19 for details. Also note the lack of synchronization for now.
arcnmx added a commit that referenced this issue Dec 20, 2020
See #19 for details. Also note the lack of synchronization for now.
arcnmx added a commit that referenced this issue Dec 20, 2020
See #19 for details. Also note the lack of synchronization for now.
arcnmx added a commit that referenced this issue Dec 20, 2020
See #19 for details. Also note the lack of synchronization for now.
arcnmx added a commit that referenced this issue Dec 20, 2020
See #19 for details. Also note the lack of synchronization for now.
@Rio6
Copy link
Rio6 commented Dec 30, 2020

Is this issue resolved?

arcnmx added a commit that referenced this issue Mar 29, 2021
See #19 for details. Also note the lack of synchronization for now.
arcnmx added a commit that referenced this issue May 21, 2021
See #19 for details. Also note the lack of synchronization for now.
arcnmx added a commit that referenced this issue May 21, 2021
See #19 for details. Also note the lack of synchronization for now.
arcnmx added a commit that referenced this issue Aug 8, 2021
See #19 for details. Also note the lack of synchronization for now.
arcnmx added a commit that referenced this issue Aug 8, 2021
See #19 for details. Also note the lack of synchronization for now.
arcnmx added a commit that referenced this issue Sep 20, 2021
See #19 for details. Also note the lack of synchronization for now.
arcnmx added a commit that referenced this issue Sep 20, 2021
See #19 for details. Also note the lack of synchronization for now.
arcnmx added a commit that referenced this issue Oct 25, 2021
See #19 for details. Also note the lack of synchronization for now.
arcnmx added a commit that referenced this issue Oct 25, 2021
See #19 for details. Also note the lack of synchronization for now.
arcnmx added a commit that referenced this issue May 28, 2022
See #19 for details. Also note the lack of synchronization for now.
arcnmx added a commit that referenced this issue Jul 18, 2022
See #19 for details. Also note the lack of synchronization for now.
arcnmx added a commit that referenced this issue Jul 18, 2022
See #19 for details. Also note the lack of synchronization for now.
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

No branches or pull requests

2 participants
0