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

	F151376305: D52753.id162890.diff
	Tue, Apr 7, 11:52 PM

	Unknown Object (File)
	Tue, Apr 7, 9:32 AM

	Unknown Object (File)
	Mon, Apr 6, 7:48 PM

	Unknown Object (File)
	Mon, Apr 6, 2:12 PM

	Unknown Object (File)
	Sat, Apr 4, 11:49 AM

	Unknown Object (File)
	Tue, Mar 31, 6:39 PM

	Unknown Object (File)
	Mon, Mar 30, 12:18 PM

	Unknown Object (File)
	Sun, Mar 22, 11:31 PM

View All 53 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 Not Applicable

Unit

Tests Not Applicable

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.Thu, Apr 2, 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 162954

lib/libc/gen/sysconf.c

Loading...

lib/libc/tests/stdtime/Makefile

Loading...

lib/libc/tests/stdtime/detect_tz_changes_test.c

Loading...