Skip to content

Add error catch for unauthorized access when removing AppLocker test files#15881

Merged
TravisEz13 merged 1 commit intoPowerShell:masterfrom
PaulHigin:fix-applocker-test
Aug 10, 2021
Merged

Add error catch for unauthorized access when removing AppLocker test files#15881
TravisEz13 merged 1 commit intoPowerShell:masterfrom
PaulHigin:fix-applocker-test

Conversation

@PaulHigin
Copy link
Copy Markdown
Contributor

PR Summary

Some anti virus applications appear to be changing access of test files written for AppLocker policy test, preventing PowerShell from successfully starting. Issue #15523.

PR Context

We already catch file IO exceptions when removing these test files, so I feel it is Ok to also catch UnauthorizedAccessException exceptions as well.

PR Checklist

@ghost ghost assigned anmenaga Aug 5, 2021
@PaulHigin PaulHigin assigned TravisEz13 and unassigned anmenaga Aug 5, 2021
@TravisEz13 TravisEz13 added BackPort-7.0.x-Consider CL-Engine Indicates that a PR should be marked as an engine change in the Change Log labels Aug 5, 2021
@TravisEz13
Copy link
Copy Markdown
Member

@PoshChan Please remind me in 23 hours

@PoshChan
Copy link
Copy Markdown
Collaborator

PoshChan commented Aug 6, 2021

@TravisEz13, this is the reminder you requested 23 hours ago

@TravisEz13 TravisEz13 changed the title Add error catch for unauthorized access when removing applocker test files Add error catch for unauthorized access when removing AppLocker test files Aug 10, 2021
@TravisEz13 TravisEz13 merged commit e2c23fc into PowerShell:master Aug 10, 2021
@PaulHigin PaulHigin deleted the fix-applocker-test branch August 10, 2021 18:04
@iSazonov iSazonov added this to the 7.2.0-preview.9 milestone Aug 11, 2021
xtqqczze pushed a commit to xtqqczze/PowerShell-PowerShell that referenced this pull request Aug 20, 2021
@ghost
Copy link
Copy Markdown

ghost commented Aug 23, 2021

🎉v7.2.0-preview.9 has been released which incorporates this pull request.:tada:

Handy links:

@ghost
Copy link
Copy Markdown

ghost commented Sep 28, 2021

🎉v7.2.0-preview.10 has been released which incorporates this pull request.:tada:

Handy links:

daxian-dbw pushed a commit to daxian-dbw/PowerShell that referenced this pull request Oct 13, 2021
daxian-dbw pushed a commit to daxian-dbw/PowerShell that referenced this pull request Oct 13, 2021
@ghost
Copy link
Copy Markdown

ghost commented Oct 14, 2021

🎉v7.0.8 has been released which incorporates this pull request.:tada:

Handy links:

@ghost
Copy link
Copy Markdown

ghost commented Oct 14, 2021

🎉v7.1.5 has been released which incorporates this pull request.:tada:

Handy links:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Backport-7.0.x-Done BackPort-7.1.x-Done Backport to 7.1.x completed CL-Engine Indicates that a PR should be marked as an engine change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants