On Wed, Mar 26, 2014 at 10:46:16PM +0000, Charles Bailey wrote:
> On Wed, Mar 26, 2014 at 05:57:41PM -0400, Jeff King wrote:
> > Hmm, so the year you got is actually: 1623969404. That still seems off
> > to me by a factor 20. I don't know if this is really worth digging into
> > that much further, but I wonder what you would get for timestamps of:
> >
> > 99999999999999999
> > 9999999999999999
> > 999999999999999
> > etc.
> >
>
> AIX goes negative at about the same time Linux and Solaris segfault:
>
> 9999999 Sun Apr 26 10:46:39 1970 -0700
> 99999999 Sat Mar 3 02:46:39 1973 -0700
> 999999999 Sat Sep 8 18:46:39 2001 -0700
> 9999999999 Sat Nov 20 10:46:39 2286 -0700
> 99999999999 Wed Nov 16 02:46:39 5138 -0700
> 999999999999 Thu Sep 26 18:46:39 33658 -0700
> 9999999999999 Sun May 20 10:46:39 318857 -0700
> 99999999999999 Sat Nov 7 02:46:39 3170843 -0700
> 999999999999999 Sat Jul 4 18:46:39 31690708 -0700
> 9999999999999999 Sat Jan 25 10:46:39 316889355 -0700
> 99999999999999999 Wed Sep 6 02:46:39 -1126091476 -0700
> 999999999999999999 Thu Oct 24 18:46:39 1623969404 -0700
Thanks. Given the value where it fails, it kind of looks like there is
some signed 32-bit value at work (~300 million years is OK, but 10 times
that, rather than yielding ~3 billion, gets us -1 billion). Perhaps
tm.tm_year is 32-bit.
So what do we want to do? I think the options are:
1. Try to guess when we have a bogus timestamp value with an arbitrary
cutoff like "greater than 1 million years from now" (and enforce it
via time_t seconds, and avoid gmtime entirely). That is made-up and
arbitrary, but it also is sufficiently far that it won't ever
matter, and sufficiently close that any gmtime should behave
sensibly with it.
2. Accept that we can't guess at every broken gmtime's output, and
just loosen the test to make sure we don't segfault.
-Peff
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html