TS file shows mostly green corruption when played by Videotoolbox

Here is the file:

https://www.dropbox.com/s/cuhqmzjq0zb9m60/sample_directv_cnn.ts?dl=0

And here is the commit that causes it:

https://github.com/videolan/vlc/commit/b6d73612d383b25ab12e2f8aad289d045200281a

Plays fine before this commit, broken ever since.

assigned to @tguillem

added Component::Decoders label

https://forum.videolan.org/viewtopic.php?f=12&t=139149&p=457462#p457462

I raised this on vlc-devel as I made a little headway in getting to the bottom of it.

https://mailman.videolan.org/pipermail/vlc-devel/2017-July/114323.html

Just an update on this one, the latest code has improved things a little in that it no longer displays green corruption. I think this is due to this commit:

e0be6843

However, this still means that on key frames it is restarting the decoder due to the issue, and according to the trace is dropping a frame each time (picture too late to be displayed).

So the underlying issue still needs solving, but the effect isn't as bad.

So, a new bugreport on the lost frame?

If you want to create a new bug report, that's fine.

Personally I can't really see the point, as you'd want to link back to this one in any case, or transfer the attachments and discussion, as it's the same underlying cause regardless of the effect on the screen.

Well, one is mostly green corruption, which seems gone, and one is "just" dropping one frame every decoder change. Or did I misunderstand?

Well it's the same bug with the same cause as discussed on the mailing list and linked to above. i.e. the decoder is getting reset apparently spuriously (I say spuriously because when I hacked it to not reset the decoder with his PPS change as I discussed on the list, it fixes the issue at least for my test file)...It's just a different effect now.

As long as you link back to this bug then I guess there is no problem with opening a new bug report!

A decoder reset shouldn't lose a frame. That is the issue, IMHO.

Please test with the newest version.

Ping with 3.0.2

Fixed for 3.0.2

added Status::fixed label

changed milestone to %3.0.x maintenance

closed

changed milestone to %3.0.7

changed milestone to %3.0.2