Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

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

Merged
TravisEz13 merged 1 commit into
PowerShell:masterPowerShell/PowerShell:masterfrom
PaulHigin:fix-applocker-testCopy head branch name to clipboard
Aug 10, 2021
Merged

Add error catch for unauthorized access when removing AppLocker test files#15881
TravisEz13 merged 1 commit into
PowerShell:masterPowerShell/PowerShell:masterfrom
PaulHigin:fix-applocker-testCopy head branch name to clipboard

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

PoshChan commented Aug 6, 2021

Copy link
Copy Markdown
Collaborator

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

Comment thread src/System.Management.Automation/utils/PathUtils.cs
@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

ghost commented Aug 23, 2021

Copy link
Copy Markdown

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

Handy links:

@ghost

ghost commented Sep 28, 2021

Copy link
Copy Markdown

🎉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

ghost commented Oct 14, 2021

Copy link
Copy Markdown

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

Handy links:

@ghost

ghost commented Oct 14, 2021

Copy link
Copy Markdown

🎉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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.