Changes between Initial Version and Version 1 of Ticket #6214, comment 5


Ignore:
Timestamp:
Mar 3, 2017, 1:19:41 AM (21 months ago)
Author:
ronag
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #6214, comment 5

    initial v1  
    11Can't say I fully understand how the dnxhd_parser works. But it looks to me that if you are unlucky you could hit a data byte that matches the prefix. https://github.com/FFmpeg/FFmpeg/blob/master/libavcodec/dnxhd_parser.c#L62 
    22 
    3 Also once {pic_found == true} don't we already know the offset to the next header? Why look for ff_dnxhd_check_header_prefix? 
     3Also once {{{pic_found == true}}} don't we already know the offset to the next header? Why look for {{{ff_dnxhd_check_header_prefix}}} for every frame?