]> git.ipfire.org Git - thirdparty/libarchive.git/commitdiff
Finish the detailed verification of zip archive with single entry for both zip64...
authorTim Kientzle <kientzle@acm.org>
Fri, 27 Dec 2013 07:00:30 +0000 (23:00 -0800)
committerTim Kientzle <kientzle@acm.org>
Fri, 27 Dec 2013 07:00:30 +0000 (23:00 -0800)
Fix a few minor issues this uncovers.

Makefile.am
libarchive/archive_write_set_format_zip.c
libarchive/test/CMakeLists.txt
libarchive/test/test_write_format_zip_file.c
libarchive/test/test_write_format_zip_file_zip64.c [new file with mode: 0644]

index 8ad0154dca1060ac7d052d7802ada126dec3c7bf..40cd09b4510b80922412219ed9c6d73a770c42e8 100644 (file)
@@ -515,6 +515,7 @@ libarchive_test_SOURCES=                                    \
        libarchive/test/test_write_format_zip_empty.c           \
        libarchive/test/test_write_format_zip_empty_zip64.c     \
        libarchive/test/test_write_format_zip_file.c            \
+       libarchive/test/test_write_format_zip_file_zip64.c      \
        libarchive/test/test_write_format_zip_compression_store.c       \
        libarchive/test/test_write_open_memory.c                \
        libarchive/test/test_zip_filename_encoding.c
index a1dc33ac93adfcb9010c5c05f4c851b8c8103554..ca3518573784692a9391082caa3123dc17716ad1 100644 (file)
@@ -473,24 +473,31 @@ archive_write_zip_header(struct archive_write *a, struct archive_entry *entry)
                if (zip->entry_compression == COMPRESSION_UNSPECIFIED) {
                        zip->entry_compression = COMPRESSION_DEFAULT;
                }
-               if (zip->force_zip64 /* User has forced it. */
-                   || zip->entry_uncompressed_size > 0xffffffffLL) /* Large entry. */
-                       zip->entry_uses_zip64 = 1;
                if (zip->entry_compression == COMPRESSION_STORE) {
                        zip->entry_compressed_size = size;
                        zip->entry_uncompressed_size = size;
                        version_needed = 10;
                } else {
+                       zip->entry_uncompressed_size = size;
                        version_needed = 20;
                }
+               if (zip->force_zip64 /* User has forced it. */
+                   || zip->entry_uncompressed_size > 0xffffffffLL) { /* Large entry. */
+                       zip->entry_uses_zip64 = 1;
+                       version_needed = 45;
+               }
+
                /* We may know the size, but never the CRC. */
                zip->entry_flags |= ZIP_FLAGS_LENGTH_AT_END;
        } else {
-               /* Prefer deflate if it's available. */
+               /* Prefer deflate if it's available, because deflate
+                * has a clear end-of-data marker that makes
+                * length-at-end more reliable. */
                zip->entry_compression = COMPRESSION_DEFAULT;
                zip->entry_flags |= ZIP_FLAGS_LENGTH_AT_END;
                if (!zip->avoid_zip64) {
                        zip->entry_uses_zip64 = 1;
+                       version_needed = 45;
                } else if (zip->entry_compression == COMPRESSION_STORE) {
                        version_needed = 10;
                } else {
@@ -498,10 +505,6 @@ archive_write_zip_header(struct archive_write *a, struct archive_entry *entry)
                }
        }
 
-       if (zip->entry_uses_zip64) {
-               version_needed = 45;
-       }
-
        /* Format the local header. */
        memset(local_header, 0, sizeof(local_header));
        memcpy(local_header, "PK\003\004", 4);
index 49f6755f42597a7daf2640c9c891e64308a6daa8..83608708b188773df9f9b8577a3d4f789c4fa30f 100644 (file)
@@ -228,6 +228,7 @@ IF(ENABLE_TEST)
     test_write_format_zip_empty.c
     test_write_format_zip_empty_zip64.c
     test_write_format_zip_file.c
+    test_write_format_zip_file_zip64.c
     test_write_format_zip_compression_store.c
     test_write_open_memory.c
     test_zip_filename_encoding.c
index a64d55d807c664b936b357f64bd3e7437b307014..eef36c6290be4afbb820929263491d32c1e06b9d 100644 (file)
 #include "test.h"
 __FBSDID("$FreeBSD: head/lib/libarchive/test/test_write_format_zip.c 201247 2009-12-30 05:59:21Z kientzle $");
 
+/*
+ * Detailed byte-for-byte verification of the format of a zip archive
+ * with a single file written to it.
+ */
+
 static unsigned long
 bitcrc32(unsigned long c, void *_p, size_t s)
 {
@@ -75,7 +80,8 @@ DEFINE_TEST(test_write_format_zip_file)
        int zip_version = 20;
        int zip_compression = 8;
        short file_uid = 10, file_gid = 20;
-       unsigned char *buff, *buffend, *p, *q, *central_header, *local_header;
+       unsigned char *buff, *buffend, *p;
+       unsigned char *central_header, *local_header, *eocd, *eocd_record;
        unsigned char *extension_start, *extension_end;
        char file_data[] = {'1', '2', '3', '4', '5', '6', '7', '8'};
        char *file_name = "file";
@@ -85,7 +91,6 @@ DEFINE_TEST(test_write_format_zip_file)
        zip_compression = 0;
 #endif
 
-
        buff = malloc(buffsize);
 
        /* Create a new archive in memory. */
@@ -111,7 +116,7 @@ DEFINE_TEST(test_write_format_zip_file)
 
        /* Verify "End of Central Directory" record. */
        /* Get address of end-of-central-directory record. */
-       p = buffend - 22; /* Assumes there is no zip comment field. */
+       eocd_record = p = buffend - 22; /* Assumes there is no zip comment field. */
        failure("End-of-central-directory begins with PK\\005\\006 signature");
        assertEqualMem(p, "PK\005\006", 4);
        failure("This must be disk 0");
@@ -120,9 +125,7 @@ DEFINE_TEST(test_write_format_zip_file)
        assertEqualInt(i2(p + 6), 0);
        failure("All central dir entries are on this disk");
        assertEqualInt(i2(p + 8), i2(p + 10));
-       failure("CD start (%d) + CD length (%d) should == archive size - 22",
-           i4(p + 12), i4(p + 16));
-       assertEqualInt(i4(p + 12) + i4(p + 16), used - 22);
+       eocd = buff + i4(p + 12) + i4(p + 16);
        failure("no zip comment");
        assertEqualInt(i2(p + 20), 0);
 
@@ -155,81 +158,84 @@ DEFINE_TEST(test_write_format_zip_file)
        p = extension_start = central_header + 46 + strlen(file_name);
        extension_end = extension_start + i2(central_header + 30);
 
+       assertEqualInt(i2(p), 0x5455);  /* 'UT' extension header */
+       assertEqualInt(i2(p + 2), 5); /* 'UT' size */
+       assertEqualInt(p[4], 1); /* 'UT' flags */
+       assertEqualInt(i4(p + 5), t); /* 'UT' mtime */
+       p += 4 + i2(p + 2);
+
+       assertEqualInt(i2(p), 0x7875);  /* 'ux' extension header */
+       assertEqualInt(i2(p + 2), 11); /* 'ux' size */
+       /* TODO: verify 'ux' contents */
+       p += 4 + i2(p + 2);
+
+       /* Just in case: Report any extra extensions. */
        while (p < extension_end) {
-               switch(i2(p)) {
-               case 0x5455:  /* 'UT' extension header */
-                       assertEqualInt(i2(p + 2), 5); /* 'UT' size */
-                       assertEqualInt(p[4], 1); /* 'UT' flags */
-                       assertEqualInt(i4(p + 5), t); /* 'UT' mtime */
-                       p = p + 4 + i2(p + 2);
-                       break;
-               case 0x7875:  /* 'ux' extension header */
-                       assertEqualInt(i2(p + 2), 11); /* 'ux' size */
-                       /* TODO: verify 'ux' contents */
-                       p = p + 4 + i2(p + 2);
-                       break;
-               default:
-                       failure("Unexpected extension 0x%04X", i2(p + 2));
-                       assert(0);
-                       break;
-               }
+               failure("Unexpected extension 0x%04X", i2(p));
+               assert(0);
+               p += 4 + i2(p + 2);
        }
+
        /* Should have run exactly to end of extra data. */
        assert(p == extension_end);
 
+       assert(p == eocd);
+
+       /* Regular EOCD immediately follows central directory. */
+       assert(p == eocd_record);
+
        /* Verify local header of file entry. */
-       q = local_header = buff;
-       assertEqualMem(q, "PK\003\004", 4); /* Signature */
-       assertEqualInt(i2(q + 4), zip_version); /* Version needed to extract */
-       assertEqualInt(i2(q + 6), 8); /* Flags */
-       assertEqualInt(i2(q + 8), zip_compression); /* Compression method */
-       assertEqualInt(i2(q + 10), (tm->tm_hour * 2048) + (tm->tm_min * 32) + (tm->tm_sec / 2)); /* File time */
-       assertEqualInt(i2(q + 12), ((tm->tm_year - 80) * 512) + ((tm->tm_mon + 1) * 32) + tm->tm_mday); /* File date */
-       assertEqualInt(i4(q + 14), 0); /* CRC-32 */
-       /* assertEqualInt(i4(q + 18), sizeof(file_data)); */ /* Compressed size */
-       /* assertEqualInt(i4(q + 22), sizeof(file_data)); */ /* Uncompressed size not stored because we're using length-at-end. */
-       assertEqualInt(i2(q + 26), strlen(file_name)); /* Pathname length */
-       assertEqualInt(i2(q + 28), 24); /* Extra field length */
-       assertEqualMem(q + 30, file_name, strlen(file_name)); /* Pathname */
-       q = extension_start = local_header + 30 + strlen(file_name);
+       p = local_header = buff;
+       assertEqualMem(p, "PK\003\004", 4); /* Signature */
+       assertEqualInt(i2(p + 4), zip_version); /* Version needed to extract */
+       assertEqualInt(i2(p + 6), 8); /* Flags */
+       assertEqualInt(i2(p + 8), zip_compression); /* Compression method */
+       assertEqualInt(i2(p + 10), (tm->tm_hour * 2048) + (tm->tm_min * 32) + (tm->tm_sec / 2)); /* File time */
+       assertEqualInt(i2(p + 12), ((tm->tm_year - 80) * 512) + ((tm->tm_mon + 1) * 32) + tm->tm_mday); /* File date */
+       assertEqualInt(i4(p + 14), 0); /* CRC-32 */
+       /* assertEqualInt(i4(p + 18), sizeof(file_data)); */ /* Compressed size */
+       /* assertEqualInt(i4(p + 22), sizeof(file_data)); */ /* Uncompressed size not stored because we're using length-at-end. */
+       assertEqualInt(i2(p + 26), strlen(file_name)); /* Pathname length */
+       assertEqualInt(i2(p + 28), 24); /* Extra field length */
+       assertEqualMem(p + 30, file_name, strlen(file_name)); /* Pathname */
+       p = extension_start = local_header + 30 + strlen(file_name);
        extension_end = extension_start + i2(local_header + 28);
 
-       while (q < extension_end) {
-               switch(i2(q)) {
-               case 0x5455:  /* 'UT' extension header */
-                       assertEqualInt(i2(q + 2), 5); /* 'UT' size */
-                       assertEqualInt(q[4], 1); /* 'UT' flags */
-                       assertEqualInt(i4(q + 5), t); /* 'UT' mtime */
-                       q = q + 4 + i2(q + 2);
-                       break;
-               case 0x7875: /* 'ux' extension header */
-                       assertEqualInt(i2(q + 2), 11); /* 'ux' size */
-                       assertEqualInt(q[4], 1); /* 'ux' version */
-                       assertEqualInt(q[5], 4); /* 'ux' uid size */
-                       assertEqualInt(i4(q + 6), file_uid); /* 'Ux' UID */
-                       assertEqualInt(q[10], 4); /* 'ux' gid size */
-                       assertEqualInt(i4(q + 11), file_gid); /* 'Ux' GID */
-                       q = q + 4 + i2(q + 2);
-                       break;
-               default:
-                       failure("Unexpected extension 0x%04X", i2(p + 2));
-                       assert(0);
-                       break;
-               }
+       assertEqualInt(i2(p), 0x5455);  /* 'UT' extension header */
+       assertEqualInt(i2(p + 2), 5); /* 'UT' size */
+       assertEqualInt(p[4], 1); /* 'UT' flags */
+       assertEqualInt(i4(p + 5), t); /* 'UT' mtime */
+       p += 4 + i2(p + 2);
+
+       assertEqualInt(i2(p), 0x7875);  /* 'ux' extension header */
+       assertEqualInt(i2(p + 2), 11); /* 'ux' size */
+       assertEqualInt(p[4], 1); /* 'ux' version */
+       assertEqualInt(p[5], 4); /* 'ux' uid size */
+       assertEqualInt(i4(p + 6), file_uid); /* 'Ux' UID */
+       assertEqualInt(p[10], 4); /* 'ux' gid size */
+       assertEqualInt(i4(p + 11), file_gid); /* 'Ux' GID */
+       p += 4 + i2(p + 2);
+
+       /* Just in case: Report any extra extensions. */
+       while (p < extension_end) {
+               failure("Unexpected extension 0x%04X", i2(p));
+               assert(0);
+               p += 4 + i2(p + 2);
        }
+
        /* Should have run exactly to end of extra data. */
-       assert(q == extension_end);
+       assert(p == extension_end);
 
        /* Data descriptor should follow compressed data. */
-       while (q < central_header && memcmp(q, "PK\007\010", 4) != 0)
-               ++q;
-       assertEqualMem(q, "PK\007\010", 4);
-       assertEqualInt(i4(q + 4), crc); /* CRC-32 */
-       /* assertEqualInt(i4(q + 8), ???); */ /* compressed size */
-       assertEqualInt(i4(q + 12), sizeof(file_data)); /* uncompressed size */
+       while (p < central_header && memcmp(p, "PK\007\010", 4) != 0)
+               ++p;
+       assertEqualMem(p, "PK\007\010", 4);
+       assertEqualInt(i4(p + 4), crc); /* CRC-32 */
+       /* assertEqualInt(i4(p + 8), ???); */ /* compressed size */
+       assertEqualInt(i4(p + 12), sizeof(file_data)); /* uncompressed size */
 
        /* Central directory should immediately follow the only entry. */
-       assert(q + 16 == central_header);
+       assert(p + 16 == central_header);
 
        free(buff);
 }
diff --git a/libarchive/test/test_write_format_zip_file_zip64.c b/libarchive/test/test_write_format_zip_file_zip64.c
new file mode 100644 (file)
index 0000000..3cc2794
--- /dev/null
@@ -0,0 +1,274 @@
+/*-
+ * Copyright (c) 2003-2008 Tim Kientzle
+ * Copyright (c) 2008 Anselm Strauss
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ *    notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ *    notice, this list of conditions and the following disclaimer in the
+ *    documentation and/or other materials provided with the distribution.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE AUTHOR(S) ``AS IS'' AND ANY EXPRESS OR
+ * IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES
+ * OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED.
+ * IN NO EVENT SHALL THE AUTHOR(S) BE LIABLE FOR ANY DIRECT, INDIRECT,
+ * INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT
+ * NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+ * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+ * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
+ * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+/*
+ * Development supported by Google Summer of Code 2008.
+ */
+
+#include "test.h"
+__FBSDID("$FreeBSD: head/lib/libarchive/test/test_write_format_zip.c 201247 2009-12-30 05:59:21Z kientzle $");
+
+/*
+ * Detailed byte-for-byte verification of the format of a zip archive
+ * with a single file written to it that uses Zip64 extensions.
+ */
+
+static unsigned long
+bitcrc32(unsigned long c, void *_p, size_t s)
+{
+       /* This is a drop-in replacement for crc32() from zlib.
+        * Libarchive should be able to correctly generate
+        * uncompressed zip archives (including correct CRCs) even
+        * when zlib is unavailable, and this function helps us verify
+        * that.  Yes, this is very, very slow and unsuitable for
+        * production use, but it's correct, compact, and works well
+        * enough for this particular usage.  Libarchive internally
+        * uses a much more efficient implementation.  */
+       const unsigned char *p = _p;
+       int bitctr;
+
+       if (p == NULL)
+               return (0);
+
+       for (; s > 0; --s) {
+               c ^= *p++;
+               for (bitctr = 8; bitctr > 0; --bitctr) {
+                       if (c & 1) c = (c >> 1);
+                       else       c = (c >> 1) ^ 0xedb88320;
+                       c ^= 0x80000000;
+               }
+       }
+       return (c);
+}
+
+/* Quick and dirty: Read 2-byte and 4-byte integers from Zip file. */
+static unsigned i2(const unsigned char *p) { return ((p[0] & 0xff) | ((p[1] & 0xff) << 8)); }
+static unsigned i4(const unsigned char *p) { return (i2(p) | (i2(p + 2) << 16)); }
+/* We're only working with small values here; ignore the 4 high bytes. */
+static unsigned i8(const unsigned char *p) { return (i4(p)); }
+
+DEFINE_TEST(test_write_format_zip_file_zip64)
+{
+       struct archive *a;
+       struct archive_entry *ae;
+       time_t t = 1234567890;
+       struct tm *tm = localtime(&t);
+       size_t used, buffsize = 1000000;
+       unsigned long crc;
+       int file_perm = 00644;
+       int zip_version = 45;
+       int zip_compression = 8;
+       short file_uid = 10, file_gid = 20;
+       unsigned char *buff, *buffend, *p;
+       unsigned char *central_header, *local_header, *eocd, *eocd_record;
+       unsigned char *extension_start, *extension_end;
+       char file_data[] = {'1', '2', '3', '4', '5', '6', '7', '8'};
+       char *file_name = "file";
+
+#ifndef HAVE_ZLIB_H
+       zip_compression = 0;
+#endif
+
+       buff = malloc(buffsize);
+
+       /* Create a new archive in memory. */
+       assert((a = archive_write_new()) != NULL);
+       assertEqualIntA(a, ARCHIVE_OK, archive_write_set_format_zip(a));
+       assertEqualIntA(a, ARCHIVE_OK, archive_write_set_options(a, "zip:zip64"));
+       assertEqualIntA(a, ARCHIVE_OK,
+           archive_write_open_memory(a, buff, buffsize, &used));
+
+       assert((ae = archive_entry_new()) != NULL);
+       archive_entry_copy_pathname(ae, file_name);
+       archive_entry_set_mode(ae, AE_IFREG | file_perm);
+       archive_entry_set_size(ae, sizeof(file_data));
+       archive_entry_set_uid(ae, file_uid);
+       archive_entry_set_gid(ae, file_gid);
+       archive_entry_set_mtime(ae, t, 0);
+       assertEqualInt(0, archive_write_header(a, ae));
+       archive_entry_free(ae);
+       assertEqualInt(8, archive_write_data(a, file_data, sizeof(file_data)));
+       assertEqualIntA(a, ARCHIVE_OK, archive_write_close(a));
+       assertEqualInt(ARCHIVE_OK, archive_write_free(a));
+       buffend = buff + used;
+       dumpfile("constructed.zip", buff, used);
+
+       /* Verify "End of Central Directory" record. */
+       /* Get address of end-of-central-directory record. */
+       eocd_record = p = buffend - 22; /* Assumes there is no zip comment field. */
+       failure("End-of-central-directory begins with PK\\005\\006 signature");
+       assertEqualMem(p, "PK\005\006", 4);
+       failure("This must be disk 0");
+       assertEqualInt(i2(p + 4), 0);
+       failure("Central dir must start on disk 0");
+       assertEqualInt(i2(p + 6), 0);
+       failure("All central dir entries are on this disk");
+       assertEqualInt(i2(p + 8), i2(p + 10));
+       eocd = buff + i4(p + 12) + i4(p + 16);
+       failure("no zip comment");
+       assertEqualInt(i2(p + 20), 0);
+
+       /* Get address of first entry in central directory. */
+       central_header = p = buff + i4(buffend - 6);
+       failure("Central file record at offset %d should begin with"
+           " PK\\001\\002 signature",
+           i4(buffend - 10));
+
+       /* Verify file entry in central directory. */
+       assertEqualMem(p, "PK\001\002", 4); /* Signature */
+       assertEqualInt(i2(p + 4), 3 * 256 + zip_version); /* Version made by */
+       assertEqualInt(i2(p + 6), zip_version); /* Version needed to extract */
+       assertEqualInt(i2(p + 8), 8); /* Flags */
+       assertEqualInt(i2(p + 10), zip_compression); /* Compression method */
+       assertEqualInt(i2(p + 12), (tm->tm_hour * 2048) + (tm->tm_min * 32) + (tm->tm_sec / 2)); /* File time */
+       assertEqualInt(i2(p + 14), ((tm->tm_year - 80) * 512) + ((tm->tm_mon + 1) * 32) + tm->tm_mday); /* File date */
+       crc = bitcrc32(0, file_data, sizeof(file_data));
+       assertEqualInt(i4(p + 16), crc); /* CRC-32 */
+       /* assertEqualInt(i4(p + 20), sizeof(file_data)); */ /* Compressed size */
+       assertEqualInt(i4(p + 24), sizeof(file_data)); /* Uncompressed size */
+       assertEqualInt(i2(p + 28), strlen(file_name)); /* Pathname length */
+       /* assertEqualInt(i2(p + 30), 28); */ /* Extra field length: See below */
+       assertEqualInt(i2(p + 32), 0); /* File comment length */
+       assertEqualInt(i2(p + 34), 0); /* Disk number start */
+       assertEqualInt(i2(p + 36), 0); /* Internal file attrs */
+       assertEqualInt(i4(p + 38) >> 16 & 01777, file_perm); /* External file attrs */
+       assertEqualInt(i4(p + 42), 0); /* Offset of local header */
+       assertEqualMem(p + 46, file_name, strlen(file_name)); /* Pathname */
+       p = extension_start = central_header + 46 + strlen(file_name);
+       extension_end = extension_start + i2(central_header + 30);
+
+       assertEqualInt(i2(p), 0x5455);  /* 'UT' extension header */
+       assertEqualInt(i2(p + 2), 5); /* 'UT' size */
+       assertEqualInt(p[4], 1); /* 'UT' flags */
+       assertEqualInt(i4(p + 5), t); /* 'UT' mtime */
+       p += 4 + i2(p + 2);
+
+       assertEqualInt(i2(p), 0x7875);  /* 'ux' extension header */
+       assertEqualInt(i2(p + 2), 11); /* 'ux' size */
+       /* TODO: verify 'ux' contents */
+       p += 4 + i2(p + 2);
+
+       /* Note: We don't expect to see zip64 extension in the central
+        * directory, since the writer knows the actual full size by
+        * the time it is ready to write the central directory and has
+        * no reason to insert it then.  Info-Zip seems to do the same
+        * thing. */
+
+       /* Just in case: Report any extra extensions. */
+       while (p < extension_end) {
+               failure("Unexpected extension 0x%04X", i2(p));
+               assert(0);
+               p += 4 + i2(p + 2);
+       }
+
+       /* Should have run exactly to end of extra data. */
+       assert(p == extension_end);
+
+       assert(p == eocd);
+
+       /* After Central dir, we find Zip64 eocd and Zip64 eocd locator. */
+       assertEqualMem(p, "PK\006\006", 4); /* Zip64 eocd */
+       assertEqualInt(i8(p + 4), 44); /* We're using v1 Zip64 eocd */
+       assertEqualInt(i2(p + 12), 45); /* Written by Version 4.5 */
+       assertEqualInt(i2(p + 14), 45); /* Needs version 4.5 to extract */
+       assertEqualInt(i4(p + 16), 0); /* This is disk #0 */
+       assertEqualInt(i4(p + 20), 0); /* Dir starts on disk #0 */
+       assertEqualInt(i8(p + 24), 1); /* 1 entry on this disk */
+       assertEqualInt(i8(p + 32), 1); /* 1 entry total */
+       assertEqualInt(i8(p + 40), eocd - central_header); /* size of cd */
+       assertEqualInt(i8(p + 48), central_header - buff); /* start of cd */
+       p += 12 + i8(p + 4);
+
+       assertEqualMem(p, "PK\006\007", 4); /* Zip64 eocd locator */
+       assertEqualInt(i4(p + 4), 0); /* Zip64 eocd is on disk #0 */
+       assertEqualInt(i8(p + 8), eocd - buff); /* Offset of Zip64 eocd */
+       assertEqualInt(i4(p + 16), 1); /* 1 disk */
+       p += 20;
+
+       /* Regular EOCD immediately follows Zip64 records. */
+       assert(p == eocd_record);
+
+       /* Verify local header of file entry. */
+       p = local_header = buff;
+       assertEqualMem(p, "PK\003\004", 4); /* Signature */
+       assertEqualInt(i2(p + 4), zip_version); /* Version needed to extract */
+       assertEqualInt(i2(p + 6), 8); /* Flags */
+       assertEqualInt(i2(p + 8), zip_compression); /* Compression method */
+       assertEqualInt(i2(p + 10), (tm->tm_hour * 2048) + (tm->tm_min * 32) + (tm->tm_sec / 2)); /* File time */
+       assertEqualInt(i2(p + 12), ((tm->tm_year - 80) * 512) + ((tm->tm_mon + 1) * 32) + tm->tm_mday); /* File date */
+       assertEqualInt(i4(p + 14), 0); /* CRC-32 */
+       /* assertEqualInt(i4(p + 18), sizeof(file_data)); */ /* Compressed size */
+       /* assertEqualInt(i4(p + 22), sizeof(file_data)); */ /* Uncompressed size not stored because we're using length-at-end. */
+       assertEqualInt(i2(p + 26), strlen(file_name)); /* Pathname length */
+       assertEqualInt(i2(p + 28), 44); /* Extra field length */
+       assertEqualMem(p + 30, file_name, strlen(file_name)); /* Pathname */
+       p = extension_start = local_header + 30 + strlen(file_name);
+       extension_end = extension_start + i2(local_header + 28);
+
+       assertEqualInt(i2(p), 0x5455);  /* 'UT' extension header */
+       assertEqualInt(i2(p + 2), 5); /* 'UT' size */
+       assertEqualInt(p[4], 1); /* 'UT' flags */
+       assertEqualInt(i4(p + 5), t); /* 'UT' mtime */
+       p += 4 + i2(p + 2);
+
+       assertEqualInt(i2(p), 0x7875);  /* 'ux' extension header */
+       assertEqualInt(i2(p + 2), 11); /* 'ux' size */
+       assertEqualInt(p[4], 1); /* 'ux' version */
+       assertEqualInt(p[5], 4); /* 'ux' uid size */
+       assertEqualInt(i4(p + 6), file_uid); /* 'Ux' UID */
+       assertEqualInt(p[10], 4); /* 'ux' gid size */
+       assertEqualInt(i4(p + 11), file_gid); /* 'Ux' GID */
+       p += 4 + i2(p + 2);
+
+       assertEqualInt(i2(p), 0x0001);  /* Zip64 extension header */
+       assertEqualInt(i2(p + 2), 16); /* size */
+       assertEqualInt(i8(p + 4), 8); /* uncompressed file size */
+       /* compressed file size we can't verify here */
+       p += 4 + i2(p + 2);
+
+       /* Just in case: Report any extra extensions. */
+       while (p < extension_end) {
+               failure("Unexpected extension 0x%04X", i2(p));
+               assert(0);
+               p += 4 + i2(p + 2);
+       }
+
+       /* Should have run exactly to end of extra data. */
+       assert(p == extension_end);
+
+       /* Data descriptor should follow compressed data. */
+       while (p < central_header && memcmp(p, "PK\007\010", 4) != 0)
+               ++p;
+       assertEqualMem(p, "PK\007\010", 4);
+       assertEqualInt(i4(p + 4), crc); /* CRC-32 */
+       /* assertEqualInt(i8(p + 8), ???); */ /* compressed size */
+       assertEqualInt(i8(p + 16), sizeof(file_data)); /* uncompressed size */
+
+       /* Central directory should immediately follow the only entry. */
+       assert(p + 24 == central_header);
+
+       free(buff);
+}