diff options
| author | Jeff King <peff@peff.net> | 2010-07-04 11:00:17 (GMT) |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2010-07-05 18:57:07 (GMT) |
| commit | 9ba0f0334dd505f78e0374bbe857c5e202f5a778 (patch) | |
| tree | 3cc507dba882fd65ce121e7838b699441ecf9050 /t/t0006-date.sh | |
| parent | ad9d8e8f0f13e054ee30fcdcdb51973ddcfadb37 (diff) | |
| download | git-9ba0f0334dd505f78e0374bbe857c5e202f5a778.zip git-9ba0f0334dd505f78e0374bbe857c5e202f5a778.tar.gz git-9ba0f0334dd505f78e0374bbe857c5e202f5a778.tar.bz2 | |
parse_date: fix signedness in timezone calculation
When no timezone is specified, we deduce the offset by
subtracting the result of mktime from our calculated
timestamp.
However, our timestamp is stored as an unsigned integer,
meaning we perform the subtraction as unsigned. For a
negative offset, this means we wrap to a very high number,
and our numeric timezone is in the millions of hours. You
can see this bug by doing:
$ TZ=EST \
GIT_AUTHOR_DATE='2010-06-01 10:00' \
git commit -a -m foo
$ git cat-file -p HEAD | grep author
author Jeff King <peff@peff.net> 1275404416 +119304128
Instead, we should perform this subtraction as a time_t, the
same type that mktime returns.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t0006-date.sh')
| -rwxr-xr-x | t/t0006-date.sh | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/t/t0006-date.sh b/t/t0006-date.sh index 3ea4f9e..b2df4fe 100755 --- a/t/t0006-date.sh +++ b/t/t0006-date.sh @@ -39,6 +39,7 @@ check_parse 2008-02 bad check_parse 2008-02-14 bad check_parse '2008-02-14 20:30:45' '2008-02-14 20:30:45 +0000' check_parse '2008-02-14 20:30:45 -0500' '2008-02-14 20:30:45 -0500' +check_parse '2008-02-14 20:30:45' '2008-02-14 20:30:45 -0500' EST check_approxidate() { echo "$1 -> $2 +0000" >expect |