Skip to content
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

Raise error when streaming from a removed torrent #1166

Merged
merged 1 commit into from Sep 9, 2017

Conversation

@aveday
Copy link

aveday commented Jul 29, 2017

In Node.js 8.2.1, when a torrent is removed while a file from it is being
streamed, file-stream.js exits, crashing Node with a TypeError:

VM535 file-stream.js:64 Uncaught TypeError: Cannot read property 'get' of null

This patch checks whether the torrent has been marked as destroyed, and
if so, raises the error to the file stream.

In Node.js 8.2.1, when a torrent is removed while a file from it is being
 streamed, file-stream.js exits, crashing Node with a TypeError:

VM535 file-stream.js:64 Uncaught TypeError: Cannot read property 'get' of null

This patch checks whether the torrent has been marked as destroyed, and
 if so, raises the error to the file stream.
@DiegoRBaquero DiegoRBaquero merged commit 1f80004 into webtorrent:master Sep 9, 2017
3 checks passed
3 checks passed
Node Security No known vulnerabilities found
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@lock lock bot locked as resolved and limited conversation to collaborators May 9, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.