1berry/ruby
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

[Bug #18264] Fix memory leak in TracePoint

Browse Source 
TracePoint leaks memory because it allocates a `rb_tp_t` struct
without ever freeing it (it is created with `RUBY_TYPED_NEVER_FREE`).
This commit is contained in:
Peter Zhu 2021-10-22 14:03:23 -04:00
parent f180fa6ca3
commit a4d5ee4f31
Notes: git 2021-10-27 02:11:55 +09:00 
Merged: https://github.com/ruby/ruby/pull/5008
2 changed files with 11 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
test/ruby/test_settracefunc.rb
View File

@ -564,6 +564,16 @@ class TestSetTraceFunc < Test::Unit::TestCase
} }
end end
# Bug #18264
def test_tracpoint_memory_leak
assert_no_memory_leak([], <<-PREP, <<-CODE, rss: true)
code = proc { TracePoint.new(:line) { } }
1_000.times(&code)
PREP
1_000_000.times(&code)
CODE
end
def trace_by_set_trace_func def trace_by_set_trace_func
events = [] events = []
trace = nil trace = nil

2
vm_trace.c
View File

@ -748,7 +748,7 @@ tp_memsize(const void *ptr)
static const rb_data_type_t tp_data_type = { static const rb_data_type_t tp_data_type = {
"tracepoint", "tracepoint",
{tp_mark, RUBY_TYPED_NEVER_FREE, tp_memsize,}, {tp_mark, RUBY_TYPED_DEFAULT_FREE, tp_memsize,},
0, 0, RUBY_TYPED_FREE_IMMEDIATELY 0, 0, RUBY_TYPED_FREE_IMMEDIATELY
}; };