Skip to content

[release/2.7] Validate guest capability strings - #41569

Merged
Ben Hillis (benhillis) merged 1 commit into
release/2.7from
backport-message-buffer-validation-release-2.7
Sep 11, 2026
Merged

Ben Hillis (benhillis) merged 1 commit into
release/2.7from
backport-message-buffer-validation-release-2.7

Conversation

@benhillis

Copy link
Copy Markdown
Member

Summary

The WSLC disk-path portion does not apply because that component is not present on this release branch.

Testing

  • full x64 Debug build

Co-authored-by: Copilot <[email protected]>

Copilot-Session: 3086bdaf-bc43-4fed-88d1-3a95a21fd14e
Copilot AI lite review requested due to automatic review settings September 10, 2026 23:08
@benhillis
Ben Hillis (benhillis) requested a review from a team as a code owner September 10, 2026 23:08

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.

🟡 Changes recommended

Regression coverage for malformed or truncated guest-capabilities Buffer is still required.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Backports bounded validation of guest capability strings and kernel version parsing for release/2.7.

Changes:

  • Captures the received message span.
  • Validates and parses the kernel version within that span.
  • Updates parsing error diagnostics.
File summaries
File Reviewed changes Final review status
src/windows/service/exe/WslCoreVm.cpp Bounds guest capability string extraction and kernel version parsing. Critical (1 vote): Add regression coverage for unterminated or truncated Buffer messages.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Lite

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

Comment thread src/windows/service/exe/WslCoreVm.cpp
@benhillis
Ben Hillis (benhillis) enabled auto-merge (squash) September 10, 2026 23:36
@benhillis
Ben Hillis (benhillis) merged commit 372dd34 into release/2.7 Sep 11, 2026
8 checks passed
@benhillis
Ben Hillis (benhillis) deleted the backport-message-buffer-validation-release-2.7 branch September 11, 2026 01:41
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.

3 participants