Skip to content

Conversation

@PaulHigin
Copy link
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
Member

@PoshChan Please remind me in 23 hours

@PoshChan
Copy link
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

ghost commented Aug 23, 2021

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

Handy links:

@ghost
Copy link

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

ghost commented Oct 14, 2021

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

Handy links:

@ghost
Copy link

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