[FFmpeg-devel] [PATCH] Fix interlaced MPEG2 decoder crash (issue2367)

[Michael Niedermayer]

On Thu, Dec 23, 2010 at 07:41:11PM +0300, Anatoly Nenashev wrote:
> On 23.12.2010 18:33, Michael Niedermayer wrote:
>> On Thu, Dec 23, 2010 at 11:19:55AM +0300, Anatoly Nenashev wrote:
>>    
>>> On 23.12.2010 03:36, Michael Niedermayer wrote:
>>>      
>>>> On Thu, Dec 23, 2010 at 02:16:21AM +0300, Anatoly Nenashev wrote:
>>>>
>>>>        
>>>>> Hi!
>>>>> Full problem description available on https://roundup.ffmpeg.org/issue2367.
>>>>> After some research I've found that sample file(exploit.bin) conflicts
>>>>> with specification in the following lines:
>>>>>
>>>>>          
>>>> is anything capable of playing this file?
>>>>
>>>> or is it a damaged file?
>>>>
>>>> how has this file been created exactly?
>>>>
>>>>
>>>>        
>>> Sample file is truncated version of the original file which has been
>>> grabbed from Optelecom Siqura C-60.
>>> I've left only two first pictures to minimize the problem. Original
>>> sample uploaded to
>>> ftp://upload.ffmpeg.org/MPlayer/incoming/interlaced_mpeg2/interlaced_mpeg2.bin
>>> It can be successfully decoded by libmpeg2 tools. By the way, libmpeg2
>>> reports errors for exploit.bin and returns no frame but don't crash.
>>>      
>> If something can play this file then the patch is 100% wrong
>> if nothing can play this file then the patch is still wrong because you drop
>> the previous I field that has been decoded correctly.
>>    
>
> Yes, first field can be successfully decoded, but second can not. Who  
> will need this half decoded frame?

You are the first person ive seen who asks, who needs part of a video. Well
I guess the user who wants to watch it

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

When the tyrant has disposed of foreign enemies by conquest or treaty, and
there is nothing more to fear from them, then he is always stirring up
some war or other, in order that the people may require a leader. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20101223/cc6e570a/attachment.pgp>