diff options
-rw-r--r-- | ChangeLog | 7 | ||||
-rw-r--r-- | eval_method.ci | 7 | ||||
-rw-r--r-- | parse.y | 4 |
3 files changed, 11 insertions, 7 deletions
@@ -1,3 +1,10 @@ +Sun Sep 23 08:58:01 2007 Nobuyoshi Nakada <nobu@ruby-lang.org> + + * eval_method.ci (rb_attr): should not use alloca for unknowen size + input. [ruby-dev:31816] + + * parse.y (rb_intern_str): prevent str from optimization. + Sun Sep 23 06:16:04 2007 Nobuyoshi Nakada <nobu@ruby-lang.org> * eval_method.ci (remove_method): check for undefined method. diff --git a/eval_method.ci b/eval_method.ci index 762e05aff7..a647335ed6 100644 --- a/eval_method.ci +++ b/eval_method.ci @@ -409,10 +409,8 @@ void rb_attr(VALUE klass, ID id, int read, int write, int ex) { const char *name; - char *buf; ID attriv; int noex; - size_t len; if (!ex) { noex = NOEX_PUBLIC; @@ -439,10 +437,7 @@ rb_attr(VALUE klass, ID id, int read, int write, int ex) if (!name) { rb_raise(rb_eArgError, "argument needs to be symbol or string"); } - len = strlen(name) + 2; - buf = ALLOCA_N(char, len); - snprintf(buf, len, "@%s", name); - attriv = rb_intern(buf); + attriv = rb_intern_str(rb_sprintf("@%s", name)); if (read) { rb_add_method(klass, id, NEW_IVAR(attriv), noex); } @@ -8528,7 +8528,9 @@ rb_intern(const char *name) ID rb_intern_str(VALUE str) { - return rb_intern3(RSTRING_PTR(str), RSTRING_LEN(str), rb_enc_get(str)); + ID id = rb_intern3(RSTRING_PTR(str), RSTRING_LEN(str), rb_enc_get(str)); + RB_GC_GUARD(str); + return id; } VALUE |