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.

Sign up for GitHub

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

Jump to bottom

FLV Parser issue #3

Closed
dwbuiten opened this issue Apr 2, 2016 · 3 comments
Closed

FLV Parser issue #3

dwbuiten opened this issue Apr 2, 2016 · 3 comments

Comments

@dwbuiten
Copy link
Owner

dwbuiten commented Apr 2, 2016 

heres another (first flv file i tried)

wget http://samples.ffmpeg.org/FLV/Enigma_Principles_of_Lust-part.flv
./ffmpeg  -i Enigma_Principles_of_Lust-part.flv -f framecrc -

[flv @ 0x3846a80] video stream discovered after head already parsed
[flv @ 0x3846a80] audio stream discovered after head already parsed
Assertion avctx->codec_id != AV_CODEC_ID_NONE failed at libavcodec/parser.c:144

[...]
dwbuiten added a commit that referenced this issue Apr 2, 2016
@dwbuiten
utils: Make sure to copy all the things that mpegts can set in read_p…
d40f238 
…acket

pls don't kill me nev

Fixes #3.

Signed-off-by: Derek Buitenhuis <derek.buitenhuis@gmail.com>
@dwbuiten
Copy link
Owner Author

dwbuiten commented Apr 2, 2016

Ignore the commit referencing this, it should have said #1.

@dwbuiten
Copy link
Owner Author

dwbuiten commented Apr 2, 2016 

[00:25] <@Daemon404> all i know is that flv *unsets* NOHEADER for some reason, at some point, so the codec id is never updated
[00:25] <@Daemon404> but if i remove that code i get a ton of h264 errors
[00:25] <@Daemon404> thats as far as i got

@dwbuiten
Copy link
Owner Author

dwbuiten commented Apr 2, 2016

This will be fixed when @Nevcairiel design for parameter changes is implemented.

Nevcairiel added a commit that referenced this issue Apr 3, 2016
@Nevcairiel
avformat: allow demuxers to signal late stream changes
fb13c2d 
This allows to update the internal context on any late changes.

Fixes #1 differently.
Fixes #3.
@michaelni michaelni mentioned this issue Apr 6, 2016
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Nevcairiel @dwbuiten