• Joe Tsai's avatar
    archive/tar: fix parsePAXTime · ef8e85e8
    Joe Tsai authored
    Issues fixed:
    * Could not handle quantity of seconds greater than 1<<31 on
    32bit machines since strconv.ParseInt did not treat integers as 64b.
    * Did not handle negative timestamps properly if nanoseconds were used.
    Note that "-123.456" should result in a call to time.Unix(-123, -456000000).
    * Incorrectly allowed a '-' right after the '.' (e.g., -123.-456)
    * Did not detect invalid input after the truncation point (e.g., 123.123456789badbadbad).
    
    Note that negative timestamps are allowed by PAX, but are not guaranteed
    to be portable. See the relevant specification:
    <<<
    If pax encounters a file with a negative timestamp in copy or write mode,
    it can reject the file, substitute a non-negative timestamp, or generate
    a non-portable timestamp with a leading '-'.
    >>>
    
    Since the previous behavior already partially supported negative timestamps,
    we are bound by Go's compatibility rules to keep support for them.
    However, we should at least make sure we handle them properly.
    
    Change-Id: I5686997708bfb59110ea7981175427290be737d1
    Reviewed-on: https://go-review.googlesource.com/31441
    Run-TryBot: Joe Tsai <thebrokentoaster@gmail.com>
    TryBot-Result: Gobot Gobot <gobot@golang.org>
    Reviewed-by: 's avatarBrad Fitzpatrick <bradfitz@golang.org>
    ef8e85e8
Name
Last commit
Last update
..
testdata Loading commit data...
common.go Loading commit data...
example_test.go Loading commit data...
format.go Loading commit data...
reader.go Loading commit data...
reader_test.go Loading commit data...
stat_atim.go Loading commit data...
stat_atimespec.go Loading commit data...
stat_unix.go Loading commit data...
strconv.go Loading commit data...
strconv_test.go Loading commit data...
tar_test.go Loading commit data...
writer.go Loading commit data...
writer_test.go Loading commit data...