Page MenuHomeFreeBSD
Differential D52241

tzcode: Add a test case for plain issetugid case
ClosedPublic
Actions

Authored by des on Aug 29 2025, 4:09 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Sep 1, 12:07 PM
Unknown Object (File)
Tue, Sep 1, 2:01 AM
Unknown Object (File)
Mon, Aug 31, 12:44 PM
Unknown Object (File)
Mon, Aug 31, 4:50 AM
Unknown Object (File)
Sun, Aug 30, 7:57 AM
Unknown Object (File)
Fri, Aug 28, 6:56 AM
Unknown Object (File)
Wed, Aug 26, 3:38 PM
Unknown Object (File)
Wed, Aug 26, 9:57 AM
View All Files
Subscribers
imp

Details

Reviewers
philip
Commits
rGcb991fa12e6e: tzcode: Add a test case for plain issetugid case
rG016d3ec239b3: tzcode: Add a test case for plain issetugid case
Summary

This catches a bug in tzcode which incorrectly considers TZDEFAULT as
unsafe as if it came from the environment.

Also deduplicate some repeated code, and fix a missing mode in an
open(2) call with O_CREAT set.

Event: Oslo Hackathon 202508

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 66641
Build 63524: arc lint + arc unit

Event Timeline

des created this revision.Aug 29 2025, 4:09 PM
Herald added a subscriber: imp. · View Herald TranscriptAug 29 2025, 4:09 PM
des requested review of this revision.Aug 29 2025, 4:09 PM
Harbormaster completed remote builds in B66641: Diff 161196.Aug 29 2025, 4:09 PM
des added a parent revision: D52240: tzcode: Don't treat TZDEFAULT as tainted.Aug 29 2025, 4:09 PM
philip added a comment.Aug 30 2025, 3:51 AM

Hmm...

  • Check failed: /home/philip/freebsd/src/lib/libc/tests/stdtime/detect_tz_changes_test.c:96: expect != buf (-0700 (PDT) != +0000 (-00))
des added a comment.Aug 31 2025, 9:50 AM
In D52241#1193936, @philip wrote:

Hmm...

  • Check failed: /home/philip/freebsd/src/lib/libc/tests/stdtime/detect_tz_changes_test.c:96: expect != buf (-0700 (PDT) != +0000 (-00))

Yes, the test successfully demonstrated that your libc has the bug fixed by D52240.

philip accepted this revision.Aug 31 2025, 11:48 PM

Hah. I forgot I did a previous installworld after reviewing D52240. I am not smart.
This looks good. And the test passes with the D52240 applied.

This revision is now accepted and ready to land.Aug 31 2025, 11:48 PM
philip mentioned this in D52240: tzcode: Don't treat TZDEFAULT as tainted.Sep 1 2025, 12:29 AM
Closed by commit rG016d3ec239b3: tzcode: Add a test case for plain issetugid case (authored by des). · Explain WhySep 1 2025, 6:36 AM
This revision was automatically updated to reflect the committed changes.
des added a commit: rG016d3ec239b3: tzcode: Add a test case for plain issetugid case.
des added a commit: rGcb991fa12e6e: tzcode: Add a test case for plain issetugid case.Apr 2 2026, 11:21 AM

Revision Contents
Changeset List

PathSize
lib/
libc/
tests/
stdtime/
60 lines

Diff 161196

View Options

lib/libc/tests/stdtime/detect_tz_changes_test.c

Loading...