Skip to content

fix: Prevent VirtualFunction memory leak in native layer#1121

Merged
roflmuffin merged 3 commits intoroflmuffin:mainfrom
ELDment:main-2
Nov 17, 2025
Merged

fix: Prevent VirtualFunction memory leak in native layer#1121
roflmuffin merged 3 commits intoroflmuffin:mainfrom
ELDment:main-2

Conversation

@ELDment
Copy link
Copy Markdown
Contributor

@ELDment ELDment commented Nov 16, 2025

Prevent VirtualFunction memory leak in native layer

image 
===== Dumping leaks =====
        Vector: 0 (0 B)
        Angles: 0 (0 B)
        GameEvents: 0 (~B)
        Virtual Functions: 2 (160 B)
        Total size: 160 B
===== Dumping leaks =====

Based on the idea from #1090 and suggestions from #1090 (comment) and #1090 (comment)

@ELDment ELDment requested a review from roflmuffin as a code owner November 16, 2025 05:57
roflmuffin
roflmuffin reviewed Nov 17, 2025
View reviewed changes
roflmuffin
roflmuffin reviewed Nov 17, 2025
View reviewed changes
@ELDment ELDment requested a review from roflmuffin November 17, 2025 01:56
@roflmuffin roflmuffin enabled auto-merge (squash) November 17, 2025 01:58
@roflmuffin
Copy link
Copy Markdown
Owner

roflmuffin commented Nov 17, 2025

Thank you!

@roflmuffin roflmuffin merged commit ff29836 into roflmuffin:main Nov 17, 2025
6 checks passed
@roflmuffin roflmuffin mentioned this pull request Nov 17, 2025
Closed
@ELDment ELDment deleted the main-2 branch November 17, 2025 02:09
dollannn pushed a commit to playpark/CounterStrikeSharp that referenced this pull request Dec 6, 2025
@ELDment @dollannn
fix: prevent VirtualFunction memory leak in native layer (roflmuffin#…
16f1c24 
…1121)
This was referenced Dec 8, 2025
Merged
Merged
Closed
Closed
Closed
Closed
Closed
Closed
Merged
Closed
Closed
Merged
Closed
Closed
Merged
Merged
This was referenced Dec 15, 2025
Closed
Merged
Merged
Closed
Merged
Open
This was referenced Dec 23, 2025
Closed
Closed
Closed
This was referenced Jan 19, 2026
Closed
Closed
Closed
@dependabot dependabot bot mentioned this pull request Jan 28, 2026
Open
This was referenced Mar 12, 2026
Closed
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@roflmuffin roflmuffin roflmuffin approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ELDment @roflmuffin