Skip to content

Fix address sanitizer issues#130932

Open
janvorli wants to merge 1 commit into
dotnet:mainfrom
janvorli:fix-asan-issue
Open

Fix address sanitizer issues#130932
janvorli wants to merge 1 commit into
dotnet:mainfrom
janvorli:fix-asan-issue

Conversation

@janvorli

Copy link
Copy Markdown
Member

This fixes two issues discovered by the address sanitizer:

  • ::delete called on base class pointer is not correct, as the :: tells the compiler to use sized delete and since that one gets size of the base class and not the real type, the sanitizer complains. It is currently benign, as the sized delete ignores the size.
  • Deletion of an array via non-array delete operator (and using a wrong type)

Close #130031

This fixes two issues discovered by the address sanitizer:
* ::delete called on base class pointer is not correct, as the :: tells the compiler to
  use sized delete and since that one gets size of the base class and not the real type,
  the sanitizer complains. It is currently benign, as the sized delete ignores the size.
* Deletion of an array via non-array delete operator (and using a wrong type)
@janvorli janvorli added this to the 11.0.0 milestone Jul 16, 2026
@janvorli janvorli self-assigned this Jul 16, 2026
Copilot AI review requested due to automatic review settings July 16, 2026 22:51

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 encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 3 pipeline(s).
12 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

@janvorli
janvorli requested a review from jkotas July 16, 2026 23:11
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.

ASan new-delete-type-mismatch in stress-http (HTTP 3.0)

3 participants