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
Unknown Object (File)
Tue, Dec 10, 12:15 PM
Unknown Object (File)
Fri, Dec 6, 11:48 PM
Unknown Object (File)
Thu, Dec 5, 3:12 AM
Unknown Object (File)
Nov 26 2024, 6:16 PM
Unknown Object (File)
Nov 21 2024, 3:04 PM
Unknown Object (File)
Nov 21 2024, 12:56 PM
Unknown Object (File)
Nov 20 2024, 10:36 PM
Unknown Object (File)
Nov 19 2024, 9:32 AM
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!