vp8: always update next_framep[] before returning from decode_frame().
authorRonald S. Bultje <rsbultje@gmail.com>
Tue, 31 Jan 2012 23:17:59 +0000 (15:17 -0800)
committerRonald S. Bultje <rsbultje@gmail.com>
Tue, 7 Feb 2012 19:29:02 +0000 (11:29 -0800)
commitfb90785e98ac405198c0ca9fec133227f6d82826
tree7ff4f6462e07adf9376274a3e9061ac9bbc74196
parent722410ade56f67b474e4d0e5684e66d4942f5973
vp8: always update next_framep[] before returning from decode_frame().

Also slightly move around code not allocate a new frame if we won't
decode it. This prevents us from putting undecoded frames in frame
pointers, which (in mt decoding) other threads will use and wait on
as references, causing a deadlock (if we skipped decoding) or a crash
(if we didn't initialized next_framep[] at all).

Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
libavcodec/vp8.c