Fix Zip64 behavior in zip2zip
This was blindly copying the zip64 extra fields from the central
directory of the original zip file to the new one. The zip64 extra
fields depend on the contents of the header it's attached toa, and in
this case we were copying the zip64 file header offset from the central
directory entry into the destination local file header, which makes no
sense, especially since the offset changed in the new file.
So strip all zip64 extra entries, and we'll create them as necessary
when writing ou the new file.
Bug: 34704111
Test: zip2zip on the original target-files -> img that was broken
Test: m -j blueprint_tools (new android_test.go)
Change-Id: Ie3c0540b13d3afcf42f3d47fff319065952b126f
diff --git a/third_party/zip/android.go b/third_party/zip/android.go
index e2e46ff..f3b6055 100644
--- a/third_party/zip/android.go
+++ b/third_party/zip/android.go
@@ -32,6 +32,12 @@
fh := &fileHeader
fh.Flags |= 0x8
+ // The zip64 extras change between the Central Directory and Local File Header, while we use
+ // the same structure for both. The Local File Haeder is taken care of by us writing a data
+ // descriptor with the zip64 values. The Central Directory Entry is written by Close(), where
+ // the zip64 extra is automatically created and appended when necessary.
+ fh.Extra = stripZip64Extras(fh.Extra)
+
h := &header{
FileHeader: fh,
offset: uint64(w.cw.count),
@@ -70,6 +76,29 @@
return err
}
+// Strip any Zip64 extra fields
+func stripZip64Extras(input []byte) []byte {
+ ret := []byte{}
+
+ for len(input) >= 4 {
+ r := readBuf(input)
+ tag := r.uint16()
+ size := r.uint16()
+ if int(size) > len(r) {
+ break
+ }
+ if tag != zip64ExtraId {
+ ret = append(ret, input[:4+size]...)
+ }
+ input = input[4+size:]
+ }
+
+ // Keep any trailing data
+ ret = append(ret, input...)
+
+ return ret
+}
+
// CreateCompressedHeader adds a file to the zip file using the provied
// FileHeader for the file metadata.
// It returns a Writer to which the already compressed file contents