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

[wasm] Use xmalloc/xfree for jmpbuf allocation to trigger GC properly

Browse Source 
`rb_vm_tag_jmpbuf_{init,deinit}` are safe to raise exception since the
given tag is not yet pushed to `ec->tag` or already popped from it at
the time, so `ec->tag` is always valid and it's safe to raise exception
when xmalloc fails.
This commit is contained in:
Yuta Saito 2023-11-22 15:23:04 +00:00
parent 341321f115
commit 295d648f76
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
vm_core.h
View File

@ -913,13 +913,13 @@ typedef rb_jmpbuf_t *rb_vm_tag_jmpbuf_t;
static inline void static inline void
rb_vm_tag_jmpbuf_init(rb_vm_tag_jmpbuf_t *jmpbuf) rb_vm_tag_jmpbuf_init(rb_vm_tag_jmpbuf_t *jmpbuf)
{ {
*jmpbuf = malloc(sizeof(rb_jmpbuf_t)); *jmpbuf = ruby_xmalloc(sizeof(rb_jmpbuf_t));
} }
static inline void static inline void
rb_vm_tag_jmpbuf_deinit(const rb_vm_tag_jmpbuf_t *jmpbuf) rb_vm_tag_jmpbuf_deinit(const rb_vm_tag_jmpbuf_t *jmpbuf)
{ {
free(*jmpbuf); ruby_xfree(*jmpbuf);
} }
#else #else
typedef rb_jmpbuf_t rb_vm_tag_jmpbuf_t; typedef rb_jmpbuf_t rb_vm_tag_jmpbuf_t;