Merge pull request #402 from msgpack/gc-guard

byroot · web-flow · commit 8c429cde4c1c · 2026-08-08T08:35:21.000+02:00
Packer_write_extension: guard on string being moved
diff --git a/ext/msgpack/packer_class.c b/ext/msgpack/packer_class.c
@@ -235,7 +235,7 @@ static VALUE Packer_write_extension(VALUE self, VALUE obj)
 
     VALUE rb_ext_type = RSTRUCT_GET(obj, 0);
     if(!RB_TYPE_P(rb_ext_type, T_FIXNUM)) {
-        rb_raise(rb_eRangeError, "integer %s too big to convert to `signed char'", RSTRING_PTR(rb_String(rb_ext_type)));
+        rb_raise(rb_eRangeError, "integer %"PRIsVALUE" too big to convert to `signed char'", rb_ext_type);
     }
 
     int ext_type = FIX2INT(rb_ext_type);

Original file line number	Diff line number	Diff line change
`@@ -235,7 +235,7 @@ static VALUE Packer_write_extension(VALUE self, VALUE obj)`
`235`	`235`
`236`	`236`	`VALUE rb_ext_type = RSTRUCT_GET(obj, 0);`
`237`	`237`	`if(!RB_TYPE_P(rb_ext_type, T_FIXNUM)) {`
`238`		- rb_raise(rb_eRangeError, "integer %s too big to convert to `signed char'", RSTRING_PTR(rb_String(rb_ext_type)));
	`238`	+ rb_raise(rb_eRangeError, "integer %"PRIsVALUE" too big to convert to `signed char'", rb_ext_type);
`239`	`239`	`}`
`240`	`240`
`241`	`241`	`int ext_type = FIX2INT(rb_ext_type);`