Skip to content

Rust: Add flow models for ToOwned trait - #22311

Merged
hvitved merged 1 commit into
github:mainfrom
hvitved:rust/to-owned-model
Aug 11, 2026
Merged

Rust: Add flow models for ToOwned trait#22311
hvitved merged 1 commit into
github:mainfrom
hvitved:rust/to-owned-model

Conversation

@hvitved

@hvitved hvitved commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions github-actions Bot added the Rust Pull requests that update Rust code label Aug 10, 2026
@hvitved hvitved added the no-change-note-required This PR does not need a change note label Aug 11, 2026
@hvitved
hvitved marked this pull request as ready for review August 11, 2026 07:07
@hvitved
hvitved requested a review from a team as a code owner August 11, 2026 07:07
@hvitved
hvitved requested review from geoffw0 and a balanced review from Copilot August 11, 2026 07:07

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.

Pull request overview

Adds Rust standard-library taint-flow models for ToOwned.

Changes:

  • Models to_owned return flow.
  • Models clone_into target mutation flow.
Show a summary per file
File Description
rust/ql/lib/codeql/rust/frameworks/stdlib/alloc.model.yml Adds ToOwned flow summaries.

Review details

Tip

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

  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Balanced

Comment thread rust/ql/lib/codeql/rust/frameworks/stdlib/alloc.model.yml

@geoffw0 geoffw0 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.

LGTM.

The DCA run (and the discussion that motivated this PR) proves the to_owned model. The clone_into model is sufficiently similar.

@hvitved
hvitved merged commit 6c02d1b into github:main Aug 11, 2026
17 checks passed
@hvitved
hvitved deleted the rust/to-owned-model branch August 11, 2026 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-change-note-required This PR does not need a change note Rust Pull requests that update Rust code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants