[FFmpeg-devel] [PATCH] fix various bugs in lzw decoder

Michael Niedermayer michaelni
Sun Jul 13 21:05:56 CEST 2008


On Sun, Jul 13, 2008 at 08:30:21PM +0200, Reimar D?ffinger wrote:
> Hello,
> while working on issue 530 I found a whole load of problems with the
> various lzw decoders (yes, it is long time to get rid of one of those,
> but I won't do it ;-) ).

just took me a few seconds to svn rm one
If anyone wants the functionality back he can start at the begin of the
review process with a clean patch, which uses the lzw&gif code from lavc.


> First, codesize must be _smaller_ than MAXBITS, since otherwise cursize
> can be larger than MAXBITS and thus a code returned by get_code might
> result in an access outside e.g. the prefix array.
> It might be conceptually nicer to change SIZTABLE instead, no idea.

maybe, anyway patch ok, it can always be changed to make it nicer


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

I have often repented speaking, but never of holding my tongue.
-- Xenocrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080713/e4a548a9/attachment.pgp>



More information about the ffmpeg-devel mailing list