Differential D52753

tzcode: Fix issues when TZ is an absolute path
ClosedPublic
Actions

Authored by des on Sep 26 2025, 5:57 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Sep 16, 9:07 AM

	Unknown Object (File)
	Thu, Sep 3, 10:34 PM

	Unknown Object (File)
	Thu, Sep 3, 10:10 PM

	Unknown Object (File)
	Thu, Sep 3, 9:29 AM

	Unknown Object (File)
	Tue, Sep 1, 6:23 AM

	Unknown Object (File)
	Tue, Aug 25, 4:02 PM

	Unknown Object (File)
	Thu, Aug 20, 7:57 PM

	Unknown Object (File)
	Aug 18 2026, 6:43 PM

View All 99 Files

Subscribers

Details

Reviewers

Commits

rGf792f3d2093c: tzcode: Fix issues when TZ is an absolute path
rG4048c39cfa6f: tzcode: Fix issues when TZ is an absolute path
rGdf8bc705eb04: tzcode: Fix issues when TZ is an absolute path

Summary

If TZ starts with TZDIR, strip any additional slashes so relname does not end up looking like an absolute path. For instance, TZ=/usr/share/zoneinfo//UTC should result in UTC, not /UTC.

In the setugid case, we were incorrectly passing name rather than relname to fstatat().

Modify the tz_env and tz_env_setugid test cases to exercise both of these scenarios.

Also add test cases for invalid values of TZ, which I wrote earlier but forgot to include in a5f14e4f9069.

Reported by: Paul Eggert <eggert@cs.ucla.edu>
MFC after: 3 days
Fixes: 967a49a21a27 ("Update tzcode to 2025b")
Fixes: a5f14e4f9069 ("tzcode: Use -00 only for invalid time zones")

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 67353
Build 64236: arc lint + arc unit

Event Timeline

des created this revision.Sep 26 2025, 5:57 PM

Herald added a subscriber: imp. · View Herald TranscriptSep 26 2025, 5:57 PM

des requested review of this revision.Sep 26 2025, 5:57 PM

Harbormaster completed remote builds in B67353: Diff 162890.Sep 26 2025, 5:57 PM

Thank you. And I'm glad to see upstream getting more in sync with us. ;-)

This revision is now accepted and ready to land.Sep 26 2025, 11:36 PM

des added a parent revision: D52694: tzcode: Drop libc namespace.Sep 27 2025, 12:16 AM

Closed by commit rGdf8bc705eb04: tzcode: Fix issues when TZ is an absolute path (authored by des). · Explain WhySep 27 2025, 5:09 PM

This revision was automatically updated to reflect the committed changes.

des added a commit: rGdf8bc705eb04: tzcode: Fix issues when TZ is an absolute path.

des added a commit: rG4048c39cfa6f: tzcode: Fix issues when TZ is an absolute path.Oct 2 2025, 8:21 AM

des added a commit: rGf792f3d2093c: tzcode: Fix issues when TZ is an absolute path.Apr 2 2026, 11:21 AM

Revision Contents
Changeset List

Path

Size

lib/

libc/

gen/

2 lines

tests/

stdtime/

1 line

detect_tz_changes_test.c

90 lines

Diff 162890

lib/libc/gen/sysconf.c

Loading...

lib/libc/tests/stdtime/Makefile

Loading...

lib/libc/tests/stdtime/detect_tz_changes_test.c

Loading...