diff options
author | Rich Felker <dalias@aerifal.cx> | 2022-02-09 17:48:43 -0500 |
---|---|---|
committer | Rich Felker <dalias@aerifal.cx> | 2022-02-09 17:48:43 -0500 |
commit | 3b7b4155570b4b9054465785be2992c92cb7d7b1 (patch) | |
tree | c6d8783133272c780b913fc30c8acc8b6b2e1f52 /include/iconv.h | |
parent | 75b3412f3dbda8f1fc6818b8b0cf1d0737c2163c (diff) | |
download | musl-3b7b4155570b4b9054465785be2992c92cb7d7b1.tar.gz musl-3b7b4155570b4b9054465785be2992c92cb7d7b1.tar.xz musl-3b7b4155570b4b9054465785be2992c92cb7d7b1.zip |
fix out-of-bound read processing time zone data with distant-past dates
this bug goes back to commit 1cc81f5cb0df2b66a795ff0c26d7bbc4d16e13c6 where zoneinfo file support was first added. in scan_trans, which searches for the appropriate local time/dst rule in effect at a given time, times prior to the second transition time caused the -1 slot of the index to be read to determine the previous rule in effect. this memory was always valid (part of another zoneinfo table in the mapped file) but the byte value read was then used to index another table, possibly going outside the bounds of the mmap. most of the time, the result was limited to misinterpretation of the rule in effect at that time (pre-1900s), but it could produce a crash if adjacent memory was not readable. the root cause of the problem, however, was that the logic for this code path was all wrong. as documented in the comment, times before the first transition should be treated as using the lowest-numbered non-dst rule, or rule 0 if no non-dst rules exist. if the argument is in units of local time, however, the rule prior to the first transition is needed to determine if it falls before or after it, and that's where the -1 index was wrongly used. instead, use the documented logic to find out what rule would be in effect before the first transition, and apply it as the offset if the argument was given in local time. the new code has not been heavily tested, but no longer performs potentially out-of-bounds accesses, and successfully handles the 1883 transition from local mean time to central standard time in the test case the error was reported for.
Diffstat (limited to 'include/iconv.h')
0 files changed, 0 insertions, 0 deletions