diff options
| author | Peter Zhu <peter@peterzhu.ca> | 2025-10-05 17:29:30 -0400 |
|---|---|---|
| committer | git <svn-admin@ruby-lang.org> | 2025-10-06 21:10:10 +0000 |
| commit | 2a484ce3c313fa4cf050e599220d9a870c901f29 (patch) | |
| tree | 20b0ead0f0aa4b457dc05e3d5ef23ce6e7a2212c | |
| parent | bc8732b6c81c761a6ed693effb0f497f39f43ea3 (diff) | |
[ruby/prism] Free current_block_exits for the program
We need to free the current_block_exits in parse_program when we're done
with it to prevent memory leaks. This fixes the following memory leak detected
when running Ruby using `RUBY_FREE_AT_EXIT=1 ruby -nc -e "break"`:
Direct leak of 32 byte(s) in 1 object(s) allocated from:
#0 0x5bd3c5bc66c8 in realloc (miniruby+0x616c8) (BuildId: https://github.com/ruby/prism/commit/ba6a96e5a060)
#1 0x5bd3c5f91fd9 in pm_node_list_grow prism/templates/src/node.c.erb:35:40
#2 0x5bd3c5f91e9d in pm_node_list_append prism/templates/src/node.c.erb:48:9
#3 0x5bd3c6001fa0 in parse_block_exit prism/prism.c:15788:17
#4 0x5bd3c5fee155 in parse_expression_prefix prism/prism.c:19221:50
#5 0x5bd3c5fe9970 in parse_expression prism/prism.c:22235:23
#6 0x5bd3c5fe0586 in parse_statements prism/prism.c:13976:27
#7 0x5bd3c5fd6792 in parse_program prism/prism.c:22508:40
https://github.com/ruby/prism/commit/fdf9b8d24a
| -rw-r--r-- | prism/prism.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/prism/prism.c b/prism/prism.c index 22ac19b5e4..f85a69332e 100644 --- a/prism/prism.c +++ b/prism/prism.c @@ -22524,9 +22524,10 @@ parse_program(pm_parser_t *parser) { statements = wrap_statements(parser, statements); } else { flush_block_exits(parser, previous_block_exits); - pm_node_list_free(¤t_block_exits); } + pm_node_list_free(¤t_block_exits); + // If this is an empty file, then we're still going to parse all of the // statements in order to gather up all of the comments and such. Here we'll // correct the location information. |