commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From sebb <seb...@gmail.com>
Subject Re: [compress] Tika test error with IMPLODEd zip
Date Sun, 22 Dec 2013 10:48:03 GMT
On 22 December 2013 07:45, Stefan Bodewig <bodewig@apache.org> wrote:
> Hi
>
> even thouhg Gump stopped nagging, it still works.  Tika is built by Gump
> and they have a test they call testUnsupportedZipCompressionMethod in
> which they seem to use an archive created with an IMPLODED entry.
>
> This test started to fail when Emmanuel added support for IMPLODE but
> unfortunately it doesn't fail because we could now read the archive but
> rather because of an IndexOutOfBounds in BinaryTree
>
> <http://vmgump.apache.org/gump/public/tika/tika-parsers-test/gump_file/org.apache.tika.parser.pkg.ZipParserTest.txt.html>
>
> Of course it is possible the archive isn't real and they used a
> hex-editor to have an unparseable archive - but it may also be a sign we
> have a bug in our code.

Seems to me that the code should still not fail with exceptions such
as IOOB (or NPE etc) no matter what is thrown at it.
It should really detect the malformation and report it with a more
suitable exception.

> I'll try to locate the archive they use later and see whether it looks
> genuine but maybe somebody from tika is lurking here and can shed some
> light.
>
> Stefan
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Mime
View raw message