search

bakape / thumbnailer

Go media thumbnailer
MIT License
152 stars 36 forks source link

ffmpeg: Invalid data found when processing input #50

Closed bakape closed 5 years ago

bakape commented 5 years ago

Triggered by: bd010adbacde6e9129e51b3bc2aaf80e

Chiiruno commented 5 years ago

Relevant lines: https://github.com/bakape/thumbnailer/blob/master/ffmpeg.c#L28-L31

Chiiruno commented 5 years ago

https://stackoverflow.com/questions/21549829/ffmpeg-avformat-open-input-returned-error

bakape commented 5 years ago

That's a completely different error code though.

On Mon, 25 Feb 2019, 09:19 チルノ, notifications@github.com wrote:

https://stackoverflow.com/questions/21549829/ffmpeg-avformat-open-input-returned-error

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/bakape/thumbnailer/issues/50#issuecomment-466898170, or mute the thread https://github.com/notifications/unsubscribe-auth/AHfPsKhJQy7oFbAe_hftrejJDFcU_jGeks5vQ45_gaJpZM4a_fkL .

Chiiruno commented 5 years ago

WHAT DID YOU EXPECT WE NEED BETTER DEVS

bakape commented 5 years ago

@Chiiruno Interestingly ffmpeg -i 52911933-0f601280-32b3-11e9-8049-895f79a0404c.jpg test.png works just fine with some warnings. Could you inspect what is done differently there or should I do it?

Chiiruno commented 5 years ago

No idea where to start, will probably save time/effort with you checking it. Right off the bat though, maybe because it's not resizing, or significantly changing the context?

bakape commented 5 years ago

From experience, it probably does some extra thing, that we do not. If what you said about it being triggered on avformat_open_input is true, then resizing has nothing to do with it and we need to inspect how other parts of the FFmpeg codebase call this function.

bakape commented 5 years ago

Idea: might be solved with format hints from MIME type detection, if this is the input opening failing automatic format detection.