Page MenuHomeFreeBSD

ktrace: Do not set args and envs as default
AbandonedPublic

Authored by jlduran on Nov 16 2024, 11:31 PM.
Tags
None
Referenced Files
F168561897: D47642.id146605.diff
Sat, Aug 29, 1:05 AM
Unknown Object (File)
Thu, Aug 27, 4:09 PM
Unknown Object (File)
Wed, Aug 26, 11:24 AM
Unknown Object (File)
Wed, Aug 26, 8:36 AM
Unknown Object (File)
Tue, Aug 25, 2:10 PM
Unknown Object (File)
Mon, Aug 24, 8:46 AM
Unknown Object (File)
Sun, Aug 23, 11:40 PM
Unknown Object (File)
Sun, Aug 23, 2:09 PM
Subscribers

Details

Reviewers
emaste
glebius
Summary

Do not define KTRFAC_ARGS and KTRFAC_ENVS as default points.

Reported by: Jenkins
Fixes: 65a4daeaf324 ("ktrace: log execve(2) arguments and environment")

Test Plan
/usr/tests # kyua test bin/sh/parameters/functional_test:mail1
bin/sh/parameters/functional_test:mail1  ->  passed  [0.013s]

Results file id is usr_tests.20241116-232707-755748
Results saved to /root/.kyua/store/results.usr_tests.20241116-232707-755748.db

1/1 passed (0 broken, 0 failed, 0 skipped)

Diff Detail

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

Event Timeline

Revision D47127 broke the test:
https://ci.freebsd.org/view/Test/job/FreeBSD-main-amd64-test/25696/testReport/junit/bin.sh.parameters/functional_test/mail1/

Due to exposing ENVS in the ktrace by default.
I think this is the fix. Otherwise, the fix will be to document that a and e are now part of the default set, and fixing the mail test.

Thank you!

Fixed in aab7f19974a27cac3c1088f486074c05bf95017a. Will submit the change in the manual separately. Thank you!