ffvp9 fails to decode some VP9 files

RESOLVED FIXED in Firefox 64

Status

()

enhancement
P2
normal
Rank:
10
RESOLVED FIXED
8 months ago
8 months ago

People

(Reporter: jya, Assigned: jya)

Tracking

unspecified
mozilla64
Points:
---

Firefox Tracking Flags

(firefox64 fixed)

Details

()

Attachments

(1 attachment)

Assignee

Description

8 months ago
Those files:
https://jyavenard.github.io/htmltests/mediatest/webm/vp9/yuv420p12.webm
https://jyavenard.github.io/htmltests/mediatest/webm/vp9/yuv422p10.webm
https://jyavenard.github.io/htmltests/mediatest/webm/vp9/yuv422p12.webm

Fail to play with ffmpeg.

They are made of a single frame.

That frame is passed to avcodec_decode_video2 (https://searchfox.org/mozilla-central/source/dom/media/platforms/ffmpeg/FFmpegVideoDecoder.cpp#219)
it will use all bytes, but doesn't return any frame.

Then the decoder is drained, feeding null/zero to avcodec_decode_video2 which returns -1 

Draining should never return -1.
Assignee

Comment 1

8 months ago
If I disable MT decoding (by setting mCodecContext->thread_count = 1)
then the first avcodec_decode_video2 returns 1
Assignee

Updated

8 months ago
Assignee: nobody → jyavenard
Rank: 10
Priority: -- → P2

Comment 4

8 months ago
Pushed by jyavenard@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/44cc1c10c530
Add missing pixel format in FFmpeg decoder r=padenot

Comment 5

8 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/44cc1c10c530
Status: NEW → RESOLVED
Last Resolved: 8 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
You need to log in before you can comment on or make changes to this bug.