Only attempt to resize strings not other objects

flori · flori · commit 167ada8da729 · 2019-04-29T15:56:14.000+02:00
diff --git a/ext/json/ext/parser/parser.c b/ext/json/ext/parser/parser.c
@@ -1658,7 +1658,7 @@ case 7:
 
     if (json->symbolize_names && json->parsing_name) {
       *result = rb_str_intern(*result);
-    } else {
+    } else if (RB_TYPE_P(*result, T_STRING)) {
       rb_str_resize(*result, RSTRING_LEN(*result));
     }
     if (cs >= JSON_string_first_final) {
diff --git a/ext/json/ext/parser/parser.rl b/ext/json/ext/parser/parser.rl
@@ -553,7 +553,7 @@ static char *JSON_parse_string(JSON_Parser *json, char *p, char *pe, VALUE *resu
 
     if (json->symbolize_names && json->parsing_name) {
       *result = rb_str_intern(*result);
-    } else {
+    } else if (RB_TYPE_P(*result, T_STRING)) {
       rb_str_resize(*result, RSTRING_LEN(*result));
     }
     if (cs >= JSON_string_first_final) {

Original file line number	Diff line number	Diff line change
`@@ -1658,7 +1658,7 @@ case 7:`
`1658`	`1658`
`1659`	`1659`	`if (json->symbolize_names && json->parsing_name) {`
`1660`	`1660`	`result = rb_str_intern(result);`
`1661`		`- } else {`
	`1661`	`+ } else if (RB_TYPE_P(*result, T_STRING)) {`
`1662`	`1662`	`rb_str_resize(result, RSTRING_LEN(result));`
`1663`	`1663`	`}`
`1664`	`1664`	`if (cs >= JSON_string_first_final) {`
Original file line number	Diff line number	Diff line change
`@@ -553,7 +553,7 @@ static char JSON_parse_string(JSON_Parser json, char p, char pe, VALUE *resu`
`553`	`553`
`554`	`554`	`if (json->symbolize_names && json->parsing_name) {`
`555`	`555`	`result = rb_str_intern(result);`
`556`		`- } else {`
	`556`	`+ } else if (RB_TYPE_P(*result, T_STRING)) {`
`557`	`557`	`rb_str_resize(result, RSTRING_LEN(result));`
`558`	`558`	`}`
`559`	`559`	`if (cs >= JSON_string_first_final) {`