diff options
-rw-r--r-- | ext/unicorn_http/unicorn_http.rl | 2 | ||||
-rw-r--r-- | test/unit/test_http_parser.rb | 18 |
2 files changed, 18 insertions, 2 deletions
diff --git a/ext/unicorn_http/unicorn_http.rl b/ext/unicorn_http/unicorn_http.rl index 6232e2c..a95224c 100644 --- a/ext/unicorn_http/unicorn_http.rl +++ b/ext/unicorn_http/unicorn_http.rl @@ -439,7 +439,7 @@ static void hp_mark(void *ptr) static VALUE HttpParser_alloc(VALUE klass) { struct http_parser *hp; - return Data_Make_Struct(klass, struct http_parser, hp_mark, NULL, hp); + return Data_Make_Struct(klass, struct http_parser, hp_mark, -1, hp); } diff --git a/test/unit/test_http_parser.rb b/test/unit/test_http_parser.rb index 0443b46..5b0ca9f 100644 --- a/test/unit/test_http_parser.rb +++ b/test/unit/test_http_parser.rb @@ -527,5 +527,21 @@ class HttpParserTest < Test::Unit::TestCase end end -end + # so we don't care about the portability of this test + # if it doesn't leak on Linux, it won't leak anywhere else + # unless your C compiler or platform is otherwise broken + LINUX_PROC_PID_STATUS = "/proc/self/status" + def test_memory_leak + match_rss = /^VmRSS:\s+(\d+)/ + if File.read(LINUX_PROC_PID_STATUS) =~ match_rss + before = $1.to_i + 1000000.times { Unicorn::HttpParser.new } + File.read(LINUX_PROC_PID_STATUS) =~ match_rss + after = $1.to_i + diff = after - before + assert(diff < 10000, "memory grew more than 10M: #{diff}") + end + end if RUBY_PLATFORM =~ /linux/ && test(?r, LINUX_PROC_PID_STATUS) + +end |