[bug #60952] NSTimeZone fail to deal with tzfile v2+

[Richard Frith-Macdonald]

Update of bug #60952 (project gnustep):

                  Status:                    None => Fixed                  
             Open/Closed:                    Open => In Test                

    _______________________________________________________

Follow-up Comment #1:

Thanks very much for bringing that issue to my attention and providing a patch
to fix the problem.

I couldn't use your patch directly (a few bugs and a lack of copyright
assignment to the FSF), but have updated NSTimeZone on the lines you suggest.
I agree with the idea of always holding transitions in 64bit form, and it
won't break anything.

The significant bugs I spotted were a failure to update the variable 'when' in
the chop() function to be 64bit (which would make transition lookups fail for
dates that won't fit in 32 bits) and allocation of memory for the internal
memory using trans_size (32bits on v1) but then unconditionally treating that
memory as if it contained 64bit values.  Neither would have shown up if you
only tested using v2+ zone info and didn't try looking up transitions for
times after the year 2038.
 
The new code is now in the git repository;  please give it a try and update
this issue to let me know how you got on with it.

Thanks again for pointing this bug out.


    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/bugs/?60952>

_______________________________________________
  Message sent via Savannah
  https://savannah.gnu.org/