- tz - lists.iana.org

Re: Bug in zic.c?
by Arthur David Olson Sept. 29, 2010

Sept. 29, 2010

That is indeed a bug--in particular, in the DOWLEQ (<=) code. The bug wasn't triggered up to now since all the distributed rules use DOWGEQ (>=). Below is a proposed fix (and slight simplification). Note that the checks are needed to ensure that rules specified in the zic input are representable by POSIX environment variables. --ado ------- zic.c ------- *** /tmp/geta23934 Wed Sep 29 08:45:01 2010 --- /tmp/getb23934 Wed Sep 29 08:45:01 2010 *************** *** 3,9 **** ** 2006-07-17 by Arthur David Olson. */ ! static char elsieid[] = "@(#)zic.c 8.22"; #include "private.h" #include "locale.h" --- 3,9 ---- ** 2006-07-17 by Arthur David Olson. */ ! static char elsieid[] = "@(#)zic.c 8.23"; #include "private.h" #include "locale.h" *************** *** 1881,1896 **** register int week; if (rp->r_dycode == DC_DOWGEQ) { ! week = 1 + rp->r_dayofmonth / DAYSPERWEEK; ! if ((week - 1) * DAYSPERWEEK + 1 != rp->r_dayofmonth) return -1; } else if (rp->r_dycode == DC_DOWLEQ) { if (rp->r_dayofmonth == len_months[1][rp->r_month]) week = 5; else { ! week = 1 + rp->r_dayofmonth / DAYSPERWEEK; ! if (week * DAYSPERWEEK - 1 != rp->r_dayofmonth) return -1; } } else return -1; /* "cannot happen" */ (void) sprintf(result, "M%d.%d.%d", --- 1881,1896 ---- register int week; if (rp->r_dycode == DC_DOWGEQ) { ! if ((rp->r_dayofmonth % DAYSPERWEEK) != 1) return -1; + week = 1 + rp->r_dayofmonth / DAYSPERWEEK; } else if (rp->r_dycode == DC_DOWLEQ) { if (rp->r_dayofmonth == len_months[1][rp->r_month]) week = 5; else { ! if ((rp->r_dayofmonth % DAYSPERWEEK) != 0) return -1; + week = rp->r_dayofmonth / DAYSPERWEEK; } } else return -1; /* "cannot happen" */ (void) sprintf(result, "M%d.%d.%d",

1 0

FW: Bug in zic.c?
by Olson, Arthur David (NIH/NCI) [E] Sept. 29, 2010

Sept. 29, 2010

I'm forwarding this message from Lei Liu, who is not on the time zone mailing list. Those of you who are on the list, please direct replies appropriately. --ado -----Original Message----- From: Liu, Lei [mailto:lei.liu2@windriver.com] Sent: Wednesday, September 29, 2010 4:17 To: tz(a)lecserver.nci.nih.gov Subject: Bug in zic.c? Hi, I have a question about the code in zic.c. In function stringrule I get: week = 1 + rp->r_dayofmonth / DAYSPERWEEK; if ((week - 1) * DAYSPERWEEK + 1 != rp->r_dayofmonth) return -1; //... week = 1 + rp->r_dayofmonth / DAYSPERWEEK; if (week * DAYSPERWEEK - 1 != rp->r_dayofmonth) return -1; What do these two if check here? The first one passes only when rp->r_dayofmonth mod 7 equals to 1. And the second one passes only when rp->r_dayofmonth mode 7 equals to 6. But IMO rp->r_dayofmonth can be arbitrary valid day number of the month. This question came up with a problem of compiling following input file: #RULE NAME FROM TO TYPE IN ON AT SAVE LTRS Rule localtime minimum maximum - Mar Sun<=14 2:00 1:0 PDT Rule localtime minimum maximum - Nov Sun<=7 2:00 0:0 UTC # ZONE NAME GMTOFF RULES/SAVE FORMAT Zone localtime 0:0 localtime %s zic prints out: "fault.txt", line 6: too many transitions?! (rule from "fault.txt", line 2) After I removed the two if check, this file can be successfully processed. Is this problem caused by a bug in zic.c or the invalid input data. Thanks. Lei

1 0

Bug in zic.c?
by Liu, Lei Sept. 29, 2010

Sept. 29, 2010

Hi, I have a question about the code in zic.c. In function stringrule I get: week = 1 + rp->r_dayofmonth / DAYSPERWEEK; if ((week - 1) * DAYSPERWEEK + 1 != rp->r_dayofmonth) return -1; //... week = 1 + rp->r_dayofmonth / DAYSPERWEEK; if (week * DAYSPERWEEK - 1 != rp->r_dayofmonth) return -1; What do these two if check here? The first one passes only when rp->r_dayofmonth mod 7 equals to 1. And the second one passes only when rp->r_dayofmonth mode 7 equals to 6. But IMO rp->r_dayofmonth can be arbitrary valid day number of the month. This question came up with a problem of compiling following input file: #RULE NAME FROM TO TYPE IN ON AT SAVE LTRS Rule localtime minimum maximum - Mar Sun<=14 2:00 1:0 PDT Rule localtime minimum maximum - Nov Sun<=7 2:00 0:0 UTC # ZONE NAME GMTOFF RULES/SAVE FORMAT Zone localtime 0:0 localtime %s zic prints out: "fault.txt", line 6: too many transitions?! (rule from "fault.txt", line 2) After I removed the two if check, this file can be successfully processed. Is this problem caused by a bug in zic.c or the invalid input data. Thanks. Lei

1 0

FW: FW: zic and zoneinfo.zip
by Olson, Arthur David (NIH/NCI) [E] Sept. 28, 2010

Sept. 28, 2010

I'm forwarding this message from Ravindra, who is not on the time zone mailing list. Those of you who are on the list, please direct replies appropriately. --ado -----Original Message----- From: Ravindra [mailto:rsahlot@yahoo.com] Sent: Monday, September 27, 2010 8:09 To: 'tz(a)elsie.nci.nih.gov'; Olson, Arthur David (NIH/NCI) [E] Subject: Re: FW: zic and zoneinfo.zip More testing on this revealed that it happens only when I use tzdata from version "2010l" on a 64-bit platform (localtime.c compiled as 64-bit code). valgrind does not report the memory error if localtime.c is compiled as 32-bit code or if I use tzdata2007k with tzcode2010l. Looks like, tzcode2010l and tzdata2010l do not get well together on 64-bit? Thanks, Ravindra --- On Sat, 9/25/10, Olson, Arthur David (NIH/NCI) [E] <olsona(a)dc37a.nci.nih.gov> wrote: > From: Olson, Arthur David (NIH/NCI) [E] <olsona(a)dc37a.nci.nih.gov> > Subject: FW: zic and zoneinfo.zip > To: "'tz(a)elsie.nci.nih.gov'" <tz(a)elsie.nci.nih.gov> > Cc: "'rsahlot(a)yahoo.com'" <rsahlot(a)yahoo.com> > Date: Saturday, September 25, 2010, 6:11 PM > I'm forwarding this message from > Ravindra, who is not on the time zone mailing list. Those of > you who are on the list, please direct replies > appropriately. > > (Note that the line numbers below are indeed different from > those in the 2010l on elsie.nci.nih.gov, and that there's no > "tz.c" file with a "tz_validate" function in 2010l on > elsie.nci.nih.gov.) > > > --ado > > -----Original Message----- > From: Ravindra [mailto:rsahlot@yahoo.com] > > Sent: Saturday, September 25, 2010 8:01 > To: Olson, Arthur David (NIH/NCI) [E] > Subject: RE: zic and zoneinfo.zip > > Thanks Arthur! > > Is there a place where I get pre-built zic binary for > Linux? > > I got the doubt because I could build 32-bit zic as well as > 64-bit zic from source code, and I was not sure whether to > build zic as a 32-bit binary or 64-bit binary. What I'm > getting from your response is I can use either version of > zic and use the generated timezone data files on either > platform. > > BTW, I'm looking at various possibilities because I'm using > the code and data from version "2010l" and I'm hitting > following memory error in valgrind log: > > ==5207== Thread 27: > ==5207== Conditional jump or move depends on uninitialised > value(s) > ==5207== at 0x1B9F988: typesequiv > (localtime.c:522) > ==5207== by 0x1B9F830: tzload > (localtime.c:493) > ==5207== by 0x1BA261B: tz_validate > (tz.c:1717) > > Below call is hitting above memory error (line numbers may > not match with original code because we have integrated it > in our code): > > 1712 struct > state state = {0}; > 1713 time_t > > now; > 1714 struct tm > tm; > 1715 > 1716 time(&now); > *** 1717 tzload(gmt, > &state, TRUE, 0); > ... > > tzload code: > ... > 486 if > (sp->timecnt > 1) { > 487 > for (i = 1; i < sp->timecnt; ++i) > 488 > if (typesequiv(sp, > sp->types[i], sp->types[0]) && > 489 > > differ_by_repeat(sp->ats[i], > sp->ats[0])) { > 490 > > sp->goback = > TRUE; > 491 > > break; > 492 > > } > **** 493 > for (i = sp->timecnt - 2; i >= 0; > --i) > 494 > if (typesequiv(sp, > sp->types[sp->timecnt - 1], > 495 > > sp->types[i]) && > 496 > > differ_by_repeat(sp->ats[sp->timecnt > - 1], > 497 > > sp->ats[i])) { > 498 > > sp->goahead = > TRUE; > 499 > > break; > 500 > } > 501 } > ... > 511 static int > 512 typesequiv(const struct state * const sp, const int a, > const int b) > 513 { > 514 register > int result; > 515 > 516 if (sp == NULL > || > 517 > a < 0 || a >= sp->typecnt || > 518 > b < 0 || b >= sp->typecnt) > 519 > result = FALSE; > 520 else { > 521 > register const struct ttinfo * ap = > &sp->ttis[a]; > *** 522 > register const struct ttinfo * bp = > &sp->ttis[b]; > 523 > result = ap->tt_gmtoff == > bp->tt_gmtoff && > 524 > ap->tt_isdst == > bp->tt_isdst && > 525 > ap->tt_ttisstd == > bp->tt_ttisstd && > 526 > ap->tt_ttisgmt == > bp->tt_ttisgmt && > 527 > > strcmp(&sp->chars[ap->tt_abbrind], > 528 > > &sp->chars[bp->tt_abbrind]) == > 0; > 529 } > 530 return result; > 531 } > > It will be great if you are aware of fix for this or if you > could help me how to proceed on this. I can explore this > further but I'm new to this code. > > Thanks in advance, > Ravindra > > --- On Fri, 9/24/10, Olson, Arthur David (NIH/NCI) [E] > <olsona(a)dc37a.nci.nih.gov> > wrote: > > > From: Olson, Arthur David (NIH/NCI) [E] <olsona(a)dc37a.nci.nih.gov> > > Subject: RE: zic and zoneinfo.zip > > To: "Ravindra" <rsahlot(a)yahoo.com> > > Date: Friday, September 24, 2010, 6:40 PM > > The current version of zic generates, > > for each time zone, a binary file that includes both > 32-bit > > and 64-bit versions of the zone's data; the binary > files > > should be usable on both 32- and 64-bit systems. > > > > --ado > > > >

3 4

FW: zic and zoneinfo.zip
by Olson, Arthur David (NIH/NCI) [E] Sept. 28, 2010

Sept. 28, 2010

I'm forwarding this message from Ravindra, who is not on the time zone mailing list. Those of you who are on the list, please direct replies appropriately. (Note that the line numbers below are indeed different from those in the 2010l on elsie.nci.nih.gov, and that there's no "tz.c" file with a "tz_validate" function in 2010l on elsie.nci.nih.gov.) --ado -----Original Message----- From: Ravindra [mailto:rsahlot@yahoo.com] Sent: Saturday, September 25, 2010 8:01 To: Olson, Arthur David (NIH/NCI) [E] Subject: RE: zic and zoneinfo.zip Thanks Arthur! Is there a place where I get pre-built zic binary for Linux? I got the doubt because I could build 32-bit zic as well as 64-bit zic from source code, and I was not sure whether to build zic as a 32-bit binary or 64-bit binary. What I'm getting from your response is I can use either version of zic and use the generated timezone data files on either platform. BTW, I'm looking at various possibilities because I'm using the code and data from version "2010l" and I'm hitting following memory error in valgrind log: ==5207== Thread 27: ==5207== Conditional jump or move depends on uninitialised value(s) ==5207== at 0x1B9F988: typesequiv (localtime.c:522) ==5207== by 0x1B9F830: tzload (localtime.c:493) ==5207== by 0x1BA261B: tz_validate (tz.c:1717) Below call is hitting above memory error (line numbers may not match with original code because we have integrated it in our code): 1712 struct state state = {0}; 1713 time_t now; 1714 struct tm tm; 1715 1716 time(&now); *** 1717 tzload(gmt, &state, TRUE, 0); ... tzload code: ... 486 if (sp->timecnt > 1) { 487 for (i = 1; i < sp->timecnt; ++i) 488 if (typesequiv(sp, sp->types[i], sp->types[0]) && 489 differ_by_repeat(sp->ats[i], sp->ats[0])) { 490 sp->goback = TRUE; 491 break; 492 } **** 493 for (i = sp->timecnt - 2; i >= 0; --i) 494 if (typesequiv(sp, sp->types[sp->timecnt - 1], 495 sp->types[i]) && 496 differ_by_repeat(sp->ats[sp->timecnt - 1], 497 sp->ats[i])) { 498 sp->goahead = TRUE; 499 break; 500 } 501 } ... 511 static int 512 typesequiv(const struct state * const sp, const int a, const int b) 513 { 514 register int result; 515 516 if (sp == NULL || 517 a < 0 || a >= sp->typecnt || 518 b < 0 || b >= sp->typecnt) 519 result = FALSE; 520 else { 521 register const struct ttinfo * ap = &sp->ttis[a]; *** 522 register const struct ttinfo * bp = &sp->ttis[b]; 523 result = ap->tt_gmtoff == bp->tt_gmtoff && 524 ap->tt_isdst == bp->tt_isdst && 525 ap->tt_ttisstd == bp->tt_ttisstd && 526 ap->tt_ttisgmt == bp->tt_ttisgmt && 527 strcmp(&sp->chars[ap->tt_abbrind], 528 &sp->chars[bp->tt_abbrind]) == 0; 529 } 530 return result; 531 } It will be great if you are aware of fix for this or if you could help me how to proceed on this. I can explore this further but I'm new to this code. Thanks in advance, Ravindra --- On Fri, 9/24/10, Olson, Arthur David (NIH/NCI) [E] <olsona(a)dc37a.nci.nih.gov> wrote: > From: Olson, Arthur David (NIH/NCI) [E] <olsona(a)dc37a.nci.nih.gov> > Subject: RE: zic and zoneinfo.zip > To: "Ravindra" <rsahlot(a)yahoo.com> > Date: Friday, September 24, 2010, 6:40 PM > The current version of zic generates, > for each time zone, a binary file that includes both 32-bit > and 64-bit versions of the zone's data; the binary files > should be usable on both 32- and 64-bit systems. > > --ado

5 8

FW: FW: Time Zone in Paris
by Olson, Arthur David (NIH/NCI) [E] Sept. 25, 2010

Sept. 25, 2010

I'm forwarding this message from Katia, who is not the time zone mailing list. Those of you who are on the list, please direct replies appropriately. --ado -----Original Message----- From: Katia [mailto:katia@astrologiainlinea.it] Sent: Saturday, September 25, 2010 6:44 To: 'Paul Eggert' Cc: tz(a)lecserver.nci.nih.gov Subject: R: FW: Time Zone in Paris Thank you for your reply and for having put my question in the mailing lists to which I will enjoy soon. The problem is that according to the downloaded version of the database on the Google site for updates http://code.google.com/p/tzdata/ in 1971 the time zone in Paris wasn't one hour ahead of UTC. Here is part of Paris file in XML version <from>1945-04-02T01:00:00Z</from> <to>1945-09-16T01:00:00Z</to> <dst>true</dst> <offset>7200</offset> <name>WEMT</name> </period> - <period> <from>1945-09-16T01:00:00Z</from> <to>1976-03-28T01:00:00Z</to> <dst>false</dst> <offset>0</offset> <name>WET</name> </period> - <period> Is that file correct ? Katia -----Messaggio originale----- Da: Paul Eggert [mailto:eggert@cs.ucla.edu] Inviato: giovedì 23 settembre 2010 06:37 A: katia(a)astrologiainlinea.it Cc: tz(a)elsie.nci.nih.gov Oggetto: Re: FW: Time Zone in Paris Katia wrote: > With regard to the time zone of Paris from 1945 to 1976 it results a > strange data that conflicts over other resources credited as Shank, > according to your database in 1971 the Time zone Paris was 0. Sorry, I don't see that problem. According to the tz database, the time zone in Paris was one hour ahead of UTC during that period. This agrees with Shanks & Pottenger 2003 (ISBN 0-035127-88-7); I just now checked my printed copy. It also agrees with the Ephemerides Astronomiques for 1998 from Bureau des Longitudes, according to email that we received on 1997-12-12 from Denis Excoffier. These two sources disagree only about exactly when this long period of UTC+1 ended. Shanks & Pottenger say that it ended on 1976-03-28 at 02:00 local time, whereas Excoffier wrote that it ended at 00:00 UTC, i.e., at 01:00 local time. Currently the tz database agrees with Excoffier, but we always welcome any more-authoritative sources, such as reliable newspapers published at the time.

2 1

FW: Time Zone in Paris
by Olson, Arthur David (NIH/NCI) [E] Sept. 25, 2010

Sept. 25, 2010

I'm forwarding this message from Katia, who is not on the time zone mailing list. Those of you who are on the list, please direct replies appropriately. --ado From: Katia [mailto:katia@astrologiainlinea.it] Sent: Wednesday, September 22, 2010 6:31 To: tz(a)lecserver.nci.nih.gov Subject: Time Zone in Paris Hello, we use your database to make astrological calculations. With regard to the time zone of Paris from 1945 to 1976 it results a strange data that conflicts over other resources credited as Shank, according to your database in 1971 the Time zone Paris was 0. Can you confirm this and show me the source? Thanks and good job Katia

4 3

zic and zoneinfo.zip
by Ravindra Sept. 24, 2010

Sept. 24, 2010

We use tzcode and tzdata in our app. We are adding support for 64-bit platforms (Windows and Linux) in our app. I would like to know the right way to generate zoneinfo.zip from tzdata2010l.tar.gz (using "zic -d" command). I have following doubts: 1. Is it ok to use same zoneinfo.zip for 32-bit as well as 64-bit platform? 2. Do I need to build two zic (32-bit and 64-bit) binaries in order to generate separate zoneinfo.zip files for 32-bit and 64-bit platforms? 3. If same zoneinfo.zip can be used for both the platforms, which zic should I use to generate the zoneinfo.zip (32-bit zic or 64-bit zic)? Could someone please clarify my doubts above? Thanks in advance, Ravindra

1 0

About China time zones
by Walter Korte Sept. 24, 2010

Sept. 24, 2010

To the maintainers of the tz database, We are using the TZ database in our navigation SW, and we have had feedback from our users in China that the time zone city names included do not make sense to local users. Neither Harbin, Shanghai nor Chungking (this is a very strange spelling, it is spelled Chongqing in Pinyin) is used in China to indicate a time zone. It would make most sense to remove these 3 entries and replace them with a single Beijing entry in the database. As HongKong and Macao are special administrative regions (SAR) in China, listing these makes sense. For the cities listed that are located in Xinjiang (Kashgar and Urumqi), these should not be listed as China only has one official time zone. But in practice Xinjiang uses its own time zone so keeping these would be better. Best regards, Walter Korte TomTom Asia

1 0

FW: Time Zone in Paris
by Francis SANTONI Sept. 23, 2010

Sept. 23, 2010

Hello, France uses 'de facto' Eastern time zone 1h. This transition has occurred by a switch to summer time in February 1940. There never was back in the winter time at the time of the meridian of Greenwich. France adopted DST in 1976 (Official Decree number 75 866 on September 19, 1975) so that some say France uses a simple summer time in winter and double summer time in summer but in practice its official time for winter is UT + 1hr East and summer time is UT +2hr East. (Sorry but I do not understand the interest of abbreviations given to time zones). Francis Santoni, Paris

1 0