diff options
| author | akr <akr@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2008-06-28 12:25:45 +0000 |
|---|---|---|
| committer | akr <akr@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2008-06-28 12:25:45 +0000 |
| commit | 2a592c65a31ed0b8da81a9ce0ad375d5c17aa8b9 (patch) | |
| tree | 6ef0adfb397248308136bd096c7dc3241b782809 /gc.c | |
| parent | 436defcc87afef2a730847c1ca54649bbbda1dcd (diff) | |
| download | ruby-2a592c65a31ed0b8da81a9ce0ad375d5c17aa8b9.tar.gz ruby-2a592c65a31ed0b8da81a9ce0ad375d5c17aa8b9.tar.xz ruby-2a592c65a31ed0b8da81a9ce0ad375d5c17aa8b9.zip | |
* include/ruby/ruby.h (struct RRegexp): new field usecnt. replace
str and len by src.
* gc.c (gc_mark_children): mark src field of regexp.
(obj_free): don't free str field.
* re.c (REG_BUSY): removed.
(rb_reg_initialize): prohibit re-initialize regexp.
(rb_reg_search): use usecnt to prevent freeing regexp currently
using. this prevents SEGV by:
r = /\A((a.)*(a.)*)*b/
r =~ "ab" + "\xc2\xa1".force_encoding("euc-jp")
t = Thread.new { r =~ "ab"*8 + "\xc2\xa1".force_encoding("utf-8")}
sleep 0.2
r =~ "ab"*8 + "\xc2\xa1".force_encoding("euc-jp")
git-svn-id: http://svn.ruby-lang.org/repos/ruby/trunk@17635 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'gc.c')
| -rw-r--r-- | gc.c | 6 |
1 files changed, 3 insertions, 3 deletions
@@ -1267,6 +1267,9 @@ gc_mark_children(rb_objspace_t *objspace, VALUE ptr, int lev) break; case T_REGEXP: + gc_mark(objspace, obj->as.regexp.src, lev); + break; + case T_FLOAT: case T_BIGNUM: break; @@ -1505,9 +1508,6 @@ obj_free(rb_objspace_t *objspace, VALUE obj) if (RANY(obj)->as.regexp.ptr) { onig_free(RANY(obj)->as.regexp.ptr); } - if (RANY(obj)->as.regexp.str) { - xfree(RANY(obj)->as.regexp.str); - } break; case T_DATA: if (DATA_PTR(obj)) { |