MemoryView: ensure reset rb_memory_view_t::obj on error

kou · kou · commit 0ed39345feb6 · 2021-06-18T09:29:46.000+09:00
diff --git a/ext/fiddle/memory_view.c b/ext/fiddle/memory_view.c
@@ -109,6 +109,7 @@ rb_fiddle_memview_initialize(VALUE obj, VALUE target)
     TypedData_Get_Struct(obj, struct memview_data, &fiddle_memview_data_type, data);
 
     if (!rb_memory_view_get(target, &data->view, 0)) {
+        data->view.obj = Qnil;
         rb_raise(rb_eArgError, "Unable to get a memory view from %+"PRIsVALUE, target);
     }
 

Original file line number	Diff line number	Diff line change
`@@ -109,6 +109,7 @@ rb_fiddle_memview_initialize(VALUE obj, VALUE target)`
`109`	`109`	`TypedData_Get_Struct(obj, struct memview_data, &fiddle_memview_data_type, data);`
`110`	`110`
`111`	`111`	`if (!rb_memory_view_get(target, &data->view, 0)) {`
	`112`	`+ data->view.obj = Qnil;`
`112`	`113`	`rb_raise(rb_eArgError, "Unable to get a memory view from %+"PRIsVALUE, target);`
`113`	`114`	`}`
`114`	`115`