]> Cypherpunks repositories - gostls13.git/commit
archive/zip: fix handling of Info-ZIP Unix extended timestamps
authorJoe Tsai <joetsai@digital-static.net>
Fri, 23 Feb 2018 23:08:11 +0000 (15:08 -0800)
committerJoe Tsai <joetsai@google.com>
Fri, 23 Feb 2018 23:48:32 +0000 (23:48 +0000)
commit9697a119e6af9fd08080e7765d8ba75d715117a6
tree02fcf94d71667909d01918a774d4e783d4531083
parent804e3e565e98e1ff3a22eb354a473225f6dc7351
archive/zip: fix handling of Info-ZIP Unix extended timestamps

The Info-ZIP Unix1 extra field is specified as such:
>>>
Value    Size   Description
-----    ----   -----------
0x5855   Short  tag for this extra block type ("UX")
TSize    Short  total data size for this block
AcTime   Long   time of last access (GMT/UTC)
ModTime  Long   time of last modification (GMT/UTC)
<<<

The previous handling was incorrect in that it read the AcTime field
instead of the ModTime field.

The test-osx.zip test unfortunately locked in the wrong behavior.
Manually parsing that ZIP file shows that the encoded MS-DOS
date and time are 0x4b5f and 0xa97d, which corresponds with a
date of 2017-10-31 21:11:58, which matches the correct mod time
(off by 1 second due to MS-DOS timestamp resolution).

Fixes #23901

Change-Id: I567824c66e8316b9acd103dbecde366874a4b7ef
Reviewed-on: https://go-review.googlesource.com/96895
Run-TryBot: Joe Tsai <joetsai@google.com>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
src/archive/zip/reader.go
src/archive/zip/reader_test.go