[FFmpeg-devel] [PATCH] lavc/tiff: add support for LZMA compression

[Michael Niedermayer]

On Sat, Aug 23, 2014 at 11:57:04AM -0300, James Almer wrote:
> On 23/08/14 8:37 AM, Hendrik Leppkes wrote:
> > On Sat, Aug 23, 2014 at 1:31 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
> >> On Fri, Aug 22, 2014 at 11:39:57PM -0300, James Almer wrote:
> >>> Derived from deflate code.
> >>> Requires liblzma.
> >>>
> >>> Signed-off-by: James Almer <jamrial at gmail.com>
> >>
> >> how can this be tested ?
> >> do you have a testcase / sample ?
> >>
> > 
> > This image should be lzma compressed:
> > https://www.flameeyes.eu/tmp/imgsrc001-lzma.tiff
> > 
> > - Hendrik
> 
> I get an assertion failure "width <= bytes_per_row" from this file in 
> tiff_unpack_strip(), before the lzma code even comes to play.
> libtiff seems to be able to handle it, though, so could open a ticket in track 
> after this gets committed (Without this patch the decoder just aborts as soon 
> as it finds it's lzma).

applied

assert fixed

thanks

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

What does censorship reveal? It reveals fear. -- Julian Assange
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140824/cd7c29a4/attachment.asc>