Skip to content

Add CET Shadow stack compatibility (#5496)#5507

Merged
guhetier merged 1 commit intorelease/2.5from
guhetier/binskim_dotnet_fix_cp_2.5
Oct 10, 2025
Merged

Add CET Shadow stack compatibility (#5496)#5507
guhetier merged 1 commit intorelease/2.5from
guhetier/binskim_dotnet_fix_cp_2.5

Conversation

@guhetier
Copy link
Collaborator

@guhetier guhetier commented Oct 9, 2025

Description

Fixes #5495 by adding the linker flag /CETCOMPAT, to advertise CET Shadow stacks supports.
Cherry-pick of #5496.

Testing

CI

Documentation

N/A

@guhetier guhetier requested a review from a team as a code owner October 9, 2025 18:11
@codecov
Copy link

codecov bot commented Oct 9, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.93%. Comparing base (29f7708) to head (6d70673).
⚠️ Report is 1 commits behind head on release/2.5.

Additional details and impacted files
@@               Coverage Diff               @@
##           release/2.5    #5507      +/-   ##
===============================================
- Coverage        87.29%   85.93%   -1.36%     
===============================================
  Files               59       59              
  Lines            18189    18189              
===============================================
- Hits             15878    15631     -247     
- Misses            2311     2558     +247     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@guhetier guhetier enabled auto-merge (squash) October 10, 2025 20:48
@guhetier guhetier merged commit 846061d into release/2.5 Oct 10, 2025
285 checks passed
@guhetier guhetier deleted the guhetier/binskim_dotnet_fix_cp_2.5 branch October 10, 2025 20:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants