admin/mruby-mruby
1
0
Fork 0
mirror of https://github.com/mruby/mruby synced 2026-06-08 16:11:16 +00:00
Code Issues Packages Projects Releases Wiki Activity

mruby-compiler: fix NODE_HEREDOC parser dump crash

Browse Source 
Replace dump_recur() with dump_str() in NODE_HEREDOC case to properly
handle cons-lists of string representations instead of AST nodes.
This fixes segmentation faults when dumping heredoc AST nodes.

Co-authored-by: Claude <noreply@anthropic.com>
This commit is contained in:
Yukihiro "Matz" Matsumoto
2025-09-26 08:39:57 +09:00
parent a49e4db4b6
commit af4df6d75d
2 changed files with 36 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
mrbgems/mruby-compiler/core/parse.y
+18 -2
View File
@@ -8343,8 +8343,24 @@ dump_node(mrb_state *mrb, node *tree, int offset)
break;
case NODE_HEREDOC:
printf("NODE_HEREDOC (<<%s):\n", ((parser_heredoc_info*)tree)->term);
dump_recur(mrb, ((parser_heredoc_info*)tree)->doc, offset+1);
printf("NODE_HEREDOC:\n");
if (heredoc_node(tree)->info.term) {
dump_prefix(offset+1, lineno);
printf("terminator: \"%s\"\n", heredoc_node(tree)->info.term);
}
if (heredoc_node(tree)->info.doc) {
dump_prefix(offset+1, lineno);
printf("body:\n");
dump_str(mrb, heredoc_node(tree)->info.doc, offset+2, lineno);
}
if (heredoc_node(tree)->info.allow_indent) {
dump_prefix(offset+1, lineno);
printf("allow_indent: true\n");
}
if (heredoc_node(tree)->info.remove_indent) {
dump_prefix(offset+1, lineno);
printf("remove_indent: true\n");
}
break;
default:
mrbgems/mruby-compiler/core/y.tab.c
+18 -2
View File
@@ -15172,8 +15172,24 @@ dump_node(mrb_state *mrb, node *tree, int offset)
break;
case NODE_HEREDOC:
printf("NODE_HEREDOC (<<%s):\n", ((parser_heredoc_info*)tree)->term);
dump_recur(mrb, ((parser_heredoc_info*)tree)->doc, offset+1);
printf("NODE_HEREDOC:\n");
if (heredoc_node(tree)->info.term) {
dump_prefix(offset+1, lineno);
printf("terminator: \"%s\"\n", heredoc_node(tree)->info.term);
}
if (heredoc_node(tree)->info.doc) {
dump_prefix(offset+1, lineno);
printf("body:\n");
dump_str(mrb, heredoc_node(tree)->info.doc, offset+2, lineno);
}
if (heredoc_node(tree)->info.allow_indent) {
dump_prefix(offset+1, lineno);
printf("allow_indent: true\n");
}
if (heredoc_node(tree)->info.remove_indent) {
dump_prefix(offset+1, lineno);
printf("remove_indent: true\n");
}
break;
default: