Michael Niedermayer <[email protected]> added the comment: On Tue, Jun 30, 2009 at 07:20:05PM +0000, Alexander Strange wrote: > svn updated and now I can't reproduce it. > > The SPS is corrupted at the end and it fails reading num_reorder_frames. > decode_vui_parameters() returns -1, but this is > ignored. Respecting it causes the entire file to fail to decode, in case > that's better: > > Index: libavcodec/h264.c > =================================================================== > --- libavcodec/h264.c (revision 19309) > +++ libavcodec/h264.c (working copy) > @@ -7278,7 +7278,8 @@ > > sps->vui_parameters_present_flag= get_bits1(&s->gb); > if( sps->vui_parameters_present_flag ) > - decode_vui_parameters(h, sps); > + if (decode_vui_parameters(h, sps)) > + goto fail;
the check should be <0 and with that its ok [...] _____________________________________________________ FFmpeg issue tracker <[email protected]> <https://roundup.ffmpeg.org/roundup/ffmpeg/issue1231> _____________________________________________________
