Movatterモバイル変換


[0]ホーム

URL:


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

Sign up
Appearance settings

Fallback to AppLocker afterWldpCanExecuteFile#24912

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged

Conversation

SeeminglyScience
Copy link
Collaborator

PR Summary

After#17247, AppLocker was no longer being checked if the Windows APIWldpCanExecuteFile was present. With the changes in this PR, fallback to AppLocker will once again occur ifWldpCanExecuteFile did not returnBlocked orAllowedConstrained.

PR Context

PR Checklist

@SeeminglyScience
Copy link
CollaboratorAuthor

/azp run

@azure-pipelinesAzure Pipelines
Copy link

Azure Pipelines successfully started running 4 pipeline(s).

@TravisEz13
Copy link
Member

/azp run

@azure-pipelinesAzure Pipelines
Copy link

Azure Pipelines successfully started running 4 pipeline(s).

@TravisEz13
Copy link
Member

/azp run powershell-ci-macos

@azure-pipelinesAzure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@TravisEz13
Copy link
Member

/azp run PowerShell-Windows-Packaging-CI

@azure-pipelinesAzure Pipelines
Copy link

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

@microsoft-github-policy-servicemicrosoft-github-policy-servicebot added the Review - NeededThe PR is being reviewed labelFeb 11, 2025

return modernMode switch
{
SystemScriptFileEnforcement.Block => SystemEnforcementMode.Enforce,
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Won't/shouldn't this not be the case as per the above and it would be better to throw the exception in the default branch?

@TravisEz13TravisEz13 added the CL-EngineIndicates that a PR should be marked as an engine change in the Change Log labelMar 11, 2025
@SeeminglyScience
Copy link
CollaboratorAuthor

/azp run

@azure-pipelinesAzure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Member

@daxian-dbwdaxian-dbw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

LGTM

@TravisEz13TravisEz13 merged commitdc0f399 intoPowerShell:masterMar 24, 2025
40 of 42 checks passed
@SeeminglyScienceSeeminglyScience deleted the fix-applocker-fallback branchMarch 24, 2025 21:13
@microsoft-github-policy-serviceMicrosoft GitHub Policy Service
Copy link
Contributor

microsoft-github-policy-servicebot commentedMar 24, 2025
edited by unfurl-linksbot
Loading

📣 Hey@SeeminglyScience, how did we do? We would love to hear your feedback with the link below! 🗣️

🔗https://aka.ms/PSRepoFeedback

@microsoft-github-policy-servicemicrosoft-github-policy-servicebot removed the Review - NeededThe PR is being reviewed labelMar 24, 2025
TravisEz13 added a commit to TravisEz13/PowerShell that referenced this pull requestMar 24, 2025
Co-authored-by: Travis Plunk <travis.plunk@microsoft.com>
pwshBot pushed a commit to pwshBot/PowerShell that referenced this pull requestApr 10, 2025
Co-authored-by: Travis Plunk <travis.plunk@microsoft.com>
Sysoiev-Yurii pushed a commit to Sysoiev-Yurii/PowerShell that referenced this pull requestMay 12, 2025
Co-authored-by: Travis Plunk <travis.plunk@microsoft.com>
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@jborean93jborean93jborean93 left review comments

@TravisEz13TravisEz13TravisEz13 left review comments

@daxian-dbwdaxian-dbwdaxian-dbw approved these changes

@adityapatwardhanadityapatwardhanAwaiting requested review from adityapatwardhan

Assignees
No one assigned
Labels
BackPort-7.4.x-DoneBackport-7.5.x-MigratedCL-EngineIndicates that a PR should be marked as an engine change in the Change Log
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

4 participants
@SeeminglyScience@TravisEz13@daxian-dbw@jborean93

[8]ページ先頭

©2009-2025 Movatter.jp