Skip to content

Update Ildasm.exe documentation with safety warning - #55974

Merged
jkoritzinsky merged 2 commits into
mainfrom
jkoritzinsky-patch-1
Sep 15, 2026
Merged

jkoritzinsky merged 2 commits into
mainfrom
jkoritzinsky-patch-1

Conversation

@jkoritzinsky

@jkoritzinsky jkoritzinsky commented Sep 15, 2026

Copy link
Copy Markdown
Member

Summary

Add warning about using Ildasm.exe with untrusted files.


Internal previews

File Preview link
docs/framework/tools/ildasm-exe-il-disassembler.md Learn preview

Build report

Add warning about using Ildasm.exe with untrusted files.
Copilot AI lite review requested due to automatic review settings September 15, 2026 17:55
@jkoritzinsky
jkoritzinsky requested review from a team and gewarren as code owners September 15, 2026 17:55
@dotnetrepoman dotnetrepoman Bot added this to the September 2026 milestone Sep 15, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

The warning should cover all supported untrusted input types before approval.

Get a fresh assessment by requesting another Copilot review.

Review tier: Lite
Findings: 1 Low severity

Open (1)
What changed in this PR

Updates Ildasm.exe documentation with a safety warning about untrusted files.

Changes:

  • Adds a warning advising use of trusted input.
File Description
docs/​framework/​tools/​ildasm-exe-il-disassembler.md Adds the Ildasm.exe safety warning.

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

Comment thread docs/framework/tools/ildasm-exe-il-disassembler.md Outdated
Clarified warning about using Ildasm.exe with untrusted files.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

@JulieLeeMSFT JulieLeeMSFT left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Approved.

@jkoritzinsky
jkoritzinsky merged commit 00e886b into main Sep 15, 2026
12 checks passed
@jkoritzinsky
jkoritzinsky deleted the jkoritzinsky-patch-1 branch September 15, 2026 21:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants