On 22 December 2013 07:45, Stefan Bodewig <bode...@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-unsubscr...@commons.apache.org > For additional commands, e-mail: dev-h...@commons.apache.org > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org
