Page MenuHomeFreeBSD
Differential D9534

localtime: return NULL if time_t out of range of struct tm
ClosedPublic
Actions

Authored by emaste on Feb 10 2017, 9:24 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 19, 6:03 PM
Unknown Object (File)
Sun, Nov 3, 1:49 AM
Unknown Object (File)
Wed, Oct 30, 8:28 AM
Unknown Object (File)
Sun, Oct 27, 1:57 PM
Unknown Object (File)
Oct 26 2024, 3:20 AM
Unknown Object (File)
Oct 24 2024, 6:09 AM
Unknown Object (File)
Oct 16 2024, 8:23 PM
Unknown Object (File)
Oct 6 2024, 12:44 AM
View All 78 Files
Subscribers
rafael.espindola_gmail.com

Details

Reviewers
bapt
Commits
rS313774: localtime: return NULL if time_t out of range of struct tm
Summary

Previously we would truncate tm.tm_year for time_ts corresponding to a year that does not fit in int.

This was fixed in tzcode in 2004 but has persisted in our tree rS2708.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

emaste updated this revision to Diff 24988.Feb 10 2017, 9:24 PM
emaste retitled this revision from to localtime: return NULL if time_t out of range of struct tm.
emaste updated this object.
emaste edited the test plan for this revision. (Show Details)
emaste added a reviewer: bapt.
emaste added a subscriber: rafael.espindola_gmail.com.

For reference, this broke the bash-static build with LLD: https://github.com/emaste/freebsd-ports/pull/20

bapt accepted this revision.Feb 12 2017, 5:44 PM
bapt edited edge metadata.
This revision is now accepted and ready to land.Feb 12 2017, 5:44 PM
Closed by commit rS313774: localtime: return NULL if time_t out of range of struct tm (authored by emaste). · Explain WhyFeb 15 2017, 3:32 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
contrib/
tzcode/
stdtime/
9 lines

Diff 25224

View Options

head/contrib/tzcode/stdtime/localtime.c

Loading...