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

BugFix Report Large file problem "ERROR: Unexpected start of file, error in tag sizes, couldn't arrive at prevTagSize=0" #4695

Closed
sunsocool opened this issue Jan 14, 2015 · 2 comments

Comments

@sunsocool
Copy link

@sunsocool sunsocool commented Jan 14, 2015

RtmpDump Version 2.4 Large File BugFix. "Unexpected start of file, error in tag sizes, couldn't arrive at prevTagSize=0"

When I using youtube-dl to download some video from website,I found that When the video file is very large like 4.48GB. RtmpDump will report error "ERROR: Unexpected start of file, error in tag sizes, couldn't arrive at prevTagSize=0"

So I download the source file,try and try again finally the project works, and then check and check again ... At last I figure out what's wrong with the program. In the last version seems trying to fix the large file problem using ftello instead of ftell .But the fseek function remains the same ,when execute fseek(file, 0, SEEK_END); size = ftello(file); Actually ftello(file) returns 0 to size.Finally I figure out the bug is that replace all fseek with fseeko to deal with large file problem.

goto this git for detail
Edit by @phihag: Snip link (see below)*

@phihag
Copy link
Contributor

@phihag phihag commented Jan 14, 2015

This looks quite suspect - why are you distributing binaries instead of code? Please submit a bug report and patch (NOT binaries) to the rtmpdump project.

@phihag phihag closed this Jan 14, 2015
@sunsocool
Copy link
Author

@sunsocool sunsocool commented Feb 15, 2015

I write a mail to the author but he didn't replay me.
The original not support parallel computing well, I recompile with this feature, the new compiled version can work very fast.
So as not every one like to recompile it by themselves, they only interest in just using it, So I recompiled it. The same time I report the bug to the rtmpdump project!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.