Inconsistency between ZipEntry size for ZipInputStream and JarInputStream

Deepak Chaudhry picture Deepak Chaudhry · Jan 23, 2012 · Viewed 7.1k times · Source

I have bunch of image files in a zip file which I am reading using ZipInputStream and Iterating over ZipEntry from an Applet.

   ZipInputStream zis = new ZipInputStream(in);
        ZipEntry ze = null;
        while ((ze = zis.getNextEntry()) != null) {
            htSizes.put(ze.getName(), new Integer((int) ze.getSize()));
            if (ze.isDirectory()) {
                continue;
            }
            int size = (int) ze.getSize();
            // -1 means unknown size.
            if (size == -1) {
                size = ((Integer) htSizes.get(ze.getName())).intValue();
            }
            byte[] b = new byte[(int) size];
            int rb = 0;
            int chunk = 0;
            while (((int) size - rb) > 0) {
                chunk = zis.read(b, rb, (int) size - rb);
                if (chunk == -1) {
                    break;
                }
                rb += chunk;
            }
            // add to internal resource hashtable
            htJarContents.put(ze.getName(), b);
        }

However when I put these images into a signed jar "ze.getSize()" is coming as -1, and image file is getting read incorrectly.

Can someone help me in this regards.

Answer

Jon Skeet picture Jon Skeet · Jan 23, 2012

Yes, -1 means the size is unknown - it's not clear why you're putting it into a map and then fetching it out again though..

Basically, if the size is unknown you should keep reading buffers until read returns -1. One simply way is to create a ByteArrayOutputStream, and keep copying from the ZipEntry to that - then once you've finished reading, just get the byte array from the ByteArrayOutputStream. It will handle any resizing necessary.