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

Fix Seeking with VC1 after 6702b01ee3598017f05374bfc7956493f790361c #260

Merged
merged 3 commits into from
Sep 4, 2017

Conversation

fritsch
Copy link
Contributor

@fritsch fritsch commented Aug 26, 2017

We found some issues with kodi when seeking through VC-1 files after above commit was merged. It seems the priv_data is not checked for being null.

You can reproduce the issue when pausing / resuming VC-1 files. The gen8_mfd change was tested working.

Debuglog with backtrace: http://sprunge.us/PPWX
Sample to reproduce: http://milhouse.libreelec.tv/other/MKV9_vc-1_6ch_ac3_dts_pcm.mkv

xhaihao referenced this pull request Aug 30, 2017
Intensity compensation was not present for B-frames, but only for P-frames. When a P-frame flags intensity compensation for its forward reference frame, all subsequent B-frames that use this reference frame as well, need to do intensity compensation as well.
@xhaihao
Copy link
Contributor

xhaihao commented Sep 4, 2017

I think GEN6 should have the same issue, could you fix this issue in gen6_mfd.c?

@fritsch
Copy link
Contributor Author

fritsch commented Sep 4, 2017

Done.

Copy link
Contributor

@xhaihao xhaihao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@xhaihao xhaihao merged commit a6039cb into intel:master Sep 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants