Skip to content

Commit

Permalink
Unset tracer environment variables when unneeded
Browse files Browse the repository at this point in the history
Previously, the tracer environment variables were set for the
current process, and for future steps, in the init action. In
certain scenarios (such as on MacOS ARM runners with System
Integrity Protection disabled), these environment variables are
not unset by the system and therefore later interfere with later
system calls.

This change unsets the environment variables for the current
process right in init. It also unsets the variables either at the
beginning of autobuild, or analyze, if autobuild has not run.
  • Loading branch information
angelapwen committed Aug 14, 2024
1 parent a93f8c2 commit 4fa88f5
Show file tree
Hide file tree
Showing 15 changed files with 71 additions and 6 deletions.
5 changes: 5 additions & 0 deletions lib/analyze-action.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion lib/analyze-action.js.map

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions lib/autobuild-action.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion lib/autobuild-action.js.map

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

5 changes: 5 additions & 0 deletions lib/environment.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion lib/environment.js.map

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

7 changes: 7 additions & 0 deletions lib/init-action.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading

0 comments on commit 4fa88f5

Please sign in to comment.