[FFmpeg-devel] [PATCH] msrle move error check

Baptiste Coudurier baptiste.coudurier
Sun May 24 09:43:45 CEST 2009


Hi Kostya,

On 5/23/2009 11:25 PM, Kostya wrote:
> On Sat, May 23, 2009 at 10:59:58PM -0700, Baptiste Coudurier wrote:
>> Hi
>>
>> Patch moves error check where it is actually an error.
>> At the end of the last line (EOL), line == 0 and --line will make it -1.
> 
> My proposal was to check two bytes of src[] in that condition - if they
> code end-of-picture, then it's fine, otherwise an error should be shown.
> There's no need to duplicate check IMO.

Good idea. Patch attached.

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
FFmpeg maintainer                                  http://www.ffmpeg.org
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: msrle_error2.patch
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090524/4985402f/attachment.txt>



More information about the ffmpeg-devel mailing list