[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Problems with kadmin date parsing in heimdal 0.6.1 on NetBSD
Mark Davies <mark@MCS.VUW.AC.NZ> writes:
> On examining the date parsing routines in kadmin/util.c, the routine
> str2time_t contains two calls to strptime(). The first parses "%Y-%m-%d"
> which works fine. The 2nd attempts to parse "%H:%M:%S" but fails because
> the first character in the string it is passed is a " ". If the format is
> changed to be " %H:%M:%S" the right thing happens.
> This problem doesn't occur on a Solaris 8 system, so it could be argued that
> the problem lies with NetBSD's strptime() routine. I'm not sure about the
> other BSD's or other systems. [My reading of posix.1-2003 strptime() indicates
> that the " " or a %n or %t is required to eat the leading space - mark]
This has been fixed in CVS.
> Also looking at the code in that routine, it is attempting to
> default to the end of the day if a time is not specified. But that
> doesn't work because the assignment of the default is done to the
> tm2 structure rather than tm.
Obviously a bug, fixed now. Thanks.
> (a) str2time_t should call tm2time() with 1 as the last parameter, which
> would mean that the date specified to kadmin would be interpreted as being
> in your local time zone and take into account daylight savings when
> converting it to a time_t; or
No, this isn't right. Times should be in UTC.
> (b) tm2time() should set tm.tm_isdst to 0 which I think would make mktime()
> ignore daylight savings time.
More like it, but how about this:
--- tm2time.c 1999/12/02 16:58:53 1.7
+++ tm2time.c 2004/05/07 09:09:33
@@ -51,7 +51,7 @@
- tm.tm_isdst = -1;
+ tm.tm_isdst = local ? -1 : 0;
t = mktime (&tm);