Skip to content
Navigation Menu
Sign in
Appearance settings
Platform
AI CODE CREATION
GitHub Copilot
Write better code with AI
GitHub Copilot app
Direct agents from issue to merge
MCP Registry
Integrate external tools
DEVELOPER WORKFLOWS
Actions
Automate any workflow
Codespaces
Instant dev environments
Issues
Plan and track work
Code Review
Manage code changes
Code Quality
Enforce quality at merge
APPLICATION SECURITY
GitHub Advanced Security
Find and fix vulnerabilities
Code security
Secure your code as you build
Secret protection
Stop leaks before they start
EXPLORE
Why GitHub
Documentation
Blog
Changelog
Marketplace
View all features
Solutions
BY COMPANY SIZE
Enterprises
Small and medium teams
Startups
Nonprofits
BY USE CASE
App Modernization
DevSecOps
DevOps
CI/CD
View all use cases
BY INDUSTRY
Healthcare
Financial services
Manufacturing
Government
View all industries
View all solutions
Resources
EXPLORE BY TOPIC
AI
Software Development
DevOps
Security
View all topics
EXPLORE BY TYPE
Customer stories
Events & webinars
Ebooks & reports
Business insights
GitHub Skills
SUPPORT & SERVICES
Documentation
Customer support
Community forum
Trust center
Partners
View all resources
Open Source
COMMUNITY
GitHub Sponsors
Fund open source developers
PROGRAMS
Security Lab
Maintainer Community
GitHub Stars
Archive Program
REPOSITORIES
Topics
Trending
Collections
Enterprise
ENTERPRISE SOLUTIONS
Enterprise platform
AI-powered developer platform
AVAILABLE ADD-ONS
GitHub Advanced Security
Enterprise-grade security features
Copilot for Business
Enterprise-grade AI features
Premium Support
Enterprise-grade 24/7 support
Pricing
Search
/
Sign in
Sign up
Appearance settings
You signed in with another tab or window.
Reload
to refresh your session.
You signed out in another tab or window.
Reload
to refresh your session.
You switched accounts on another tab or window.
Reload
to refresh your session.
Dismiss alert
{{ message }}
dotnet
/
runtime
Public
Notifications
You must be signed in to change notification settings
Fork
5.6k
Star
18.3k
Code
Issues
5k+
Pull requests
681
Discussions
Actions
Projects
Security and quality
70
Insights
Additional navigation options
Code
Issues
Pull requests
Discussions
Actions
Projects
Security and quality
Insights
Fix a memory leak in runtime interop stubs when using an array of structs of types that use old-style managed marshalers
- #93089
#93089
Merged
jkoritzinsky
merged 8 commits into
dotnet:main
dotnet/runtime:main
from
jkoritzinsky:struct-free
jkoritzinsky/runtime:struct-free
Copy head branch name to clipboard
Oct 10, 2023
Conversation
Commits
8
(8)
Checks
Files changed
Merged
Fix a memory leak in runtime interop stubs when using an array of structs of types that use old-style managed marshalers
#93089
jkoritzinsky
merged 8 commits into
dotnet:main
dotnet/runtime:main
from
jkoritzinsky:struct-free
jkoritzinsky/runtime:struct-free
Copy head branch name to clipboard
Commits
Commits on Oct 5, 2023
When we don't have a managed value in a ClearNative call, the managed poniter to the managed field with a null pointer instead of skipping the ClearNative call.
jkoritzinsky
committed
a06ae14
View commit details
Copy full SHA for a06ae14
Browse repository at this point
Fix duplicate assignment
jkoritzinsky
committed
42b8994
View commit details
Copy full SHA for 42b8994
Browse repository at this point
Commits on Oct 6, 2023
Add test using a combination of ComWrappers and built-in COM to validate lifetimes. We can't use ICustomMarshaler here as that isn't supported in field scenarios.
jkoritzinsky
committed
2baa6da
View commit details
Copy full SHA for 2baa6da
Browse repository at this point
Commits on Oct 9, 2023
We already normalize null refs early, so we don't need to normalize here.
jkoritzinsky
committed
c34d03b
View commit details
Copy full SHA for c34d03b
Browse repository at this point
Simplify test
jkoritzinsky
committed
6ff0065
View commit details
Copy full SHA for 6ff0065
Browse repository at this point
Update src/tests/Interop/ArrayMarshalling/StructArray/MarshalStructArrayTest.cs
Show description for 1a902f2
jkoritzinsky
and
AaronRobinsonMSFT
authored
1a902f2
View commit details
Copy full SHA for 1a902f2
Browse repository at this point
Rewrite test to be more reliable and add lots of comments about what we are testing and how we are testing it as it is very much not straightfoward.
jkoritzinsky
committed
a5ea5f6
View commit details
Copy full SHA for a5ea5f6
Browse repository at this point
Remove now-dead code
jkoritzinsky
committed
6dc66b1
View commit details
Copy full SHA for 6dc66b1
Browse repository at this point
You can’t perform that action at this time.