From c39e8c6e854032645b0f2b799294d3e96d066697 Mon Sep 17 00:00:00 2001 From: matz Date: Fri, 16 May 2008 18:27:01 +0000 Subject: [PATCH] * array.c (rb_ary_sort_bang): stop memory leak. [ruby-dev:34726] * re.c (rb_reg_search): need to free allocated buffer in re_register. * regexec.c (onig_region_new): more pedantic malloc check. * regexec.c (onig_region_resize): ditto. * regexec.c (STATE_CHECK_BUFF_INIT): ditto. * regexec.c (onig_region_copy): use onig_region_resize. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@16437 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- ChangeLog | 14 ++++++++++++++ array.c | 11 ++++++----- re.c | 2 ++ regexec.c | 50 +++++++++++++++++++++++++++++--------------------- version.h | 6 +++--- 5 files changed, 54 insertions(+), 29 deletions(-) diff --git a/ChangeLog b/ChangeLog index af7efa6fe3..565a20b128 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,17 @@ +Sat May 17 03:21:29 2008 Yukihiro Matsumoto + + * array.c (rb_ary_sort_bang): stop memory leak. [ruby-dev:34726] + + * re.c (rb_reg_search): need to free allocated buffer in re_register. + + * regexec.c (onig_region_new): more pedantic malloc check. + + * regexec.c (onig_region_resize): ditto. + + * regexec.c (STATE_CHECK_BUFF_INIT): ditto. + + * regexec.c (onig_region_copy): use onig_region_resize. + Fri May 16 12:48:33 2008 Yukihiro Matsumoto * math.c (to_flo): rb_Float() accepts even strings for input. diff --git a/array.c b/array.c index 631080d521..635c7c7837 100644 --- a/array.c +++ b/array.c @@ -1513,16 +1513,17 @@ rb_ary_sort_bang(VALUE ary) RBASIC(tmp)->klass = 0; ruby_qsort(RARRAY_PTR(tmp), RARRAY_LEN(tmp), sizeof(VALUE), rb_block_given_p()?sort_1:sort_2, &RBASIC(tmp)->klass); - sort_reentered(&RBASIC(tmp)->klass); - RARRAY(ary)->ptr = RARRAY(tmp)->ptr; - RARRAY(ary)->len = RARRAY(tmp)->len; - RARRAY(ary)->aux.capa = RARRAY(tmp)->aux.capa; + if (RARRAY(ary)->ptr != RARRAY(tmp)->ptr) { + xfree(RARRAY(ary)->ptr); + RARRAY(ary)->ptr = RARRAY(tmp)->ptr; + RARRAY(ary)->len = RARRAY(tmp)->len; + RARRAY(ary)->aux.capa = RARRAY(tmp)->aux.capa; + }; FL_UNSET(ary, ELTS_SHARED); RARRAY(tmp)->ptr = 0; RARRAY(tmp)->len = 0; RARRAY(tmp)->aux.capa = 0; RBASIC(tmp)->klass = RBASIC(ary)->klass; - OBJ_FREEZE(tmp); } return ary; } diff --git a/re.c b/re.c index c3be217443..4d9a26363f 100644 --- a/re.c +++ b/re.c @@ -1300,6 +1300,7 @@ rb_reg_search(VALUE re, VALUE str, int pos, int reverse) } if (!busy) FL_UNSET(re, REG_BUSY); if (result < 0) { + onig_region_free(®s, 0); if (result == ONIG_MISMATCH) { rb_backref_set(Qnil); return result; @@ -1323,6 +1324,7 @@ rb_reg_search(VALUE re, VALUE str, int pos, int reverse) } onig_region_copy(RMATCH_REGS(match), ®s); + onig_region_free(®s, 0); RMATCH(match)->str = rb_str_new4(str); RMATCH(match)->regexp = re; RMATCH(match)->rmatch->char_offset_updated = 0; diff --git a/regexec.c b/regexec.c index 47fd67140e..b9947d8f93 100644 --- a/regexec.c +++ b/regexec.c @@ -178,16 +178,34 @@ onig_region_resize(OnigRegion* region, int n) if (region->allocated == 0) { region->beg = (int* )xmalloc(n * sizeof(int)); - region->end = (int* )xmalloc(n * sizeof(int)); - - if (region->beg == 0 || region->end == 0) + if (region->beg == 0) return ONIGERR_MEMORY; + region->end = (int* )xmalloc(n * sizeof(int)); + if (region->end == 0) { + xfree(region->beg); + return ONIGERR_MEMORY; + } + region->allocated = n; } else if (region->allocated < n) { - region->beg = (int* )xrealloc(region->beg, n * sizeof(int)); - region->end = (int* )xrealloc(region->end, n * sizeof(int)); + int *tmp; + + region->allocated = 0; + tmp = (int* )xrealloc(region->beg, n * sizeof(int)); + if (tmp == 0) { + xfree(region->beg); + xfree(region->end); + return ONIGERR_MEMORY; + } + region->beg = tmp; + tmp = (int* )xrealloc(region->end, n * sizeof(int)); + if (tmp == 0) { + xfree(region->beg); + return ONIGERR_MEMORY; + } + region->end = tmp; if (region->beg == 0 || region->end == 0) return ONIGERR_MEMORY; @@ -240,7 +258,8 @@ onig_region_new(void) OnigRegion* r; r = (OnigRegion* )xmalloc(sizeof(OnigRegion)); - onig_region_init(r); + if (r) + onig_region_init(r); return r; } @@ -268,19 +287,7 @@ onig_region_copy(OnigRegion* to, OnigRegion* from) if (to == from) return; - if (to->allocated == 0) { - if (from->num_regs > 0) { - to->beg = (int* )xmalloc(RREGC_SIZE); - to->end = (int* )xmalloc(RREGC_SIZE); - to->allocated = from->num_regs; - } - } - else if (to->allocated < from->num_regs) { - to->beg = (int* )xrealloc(to->beg, RREGC_SIZE); - to->end = (int* )xrealloc(to->end, RREGC_SIZE); - to->allocated = from->num_regs; - } - + onig_region_resize(to, from->num_regs); for (i = 0; i < from->num_regs; i++) { to->beg[i] = from->beg[i]; to->end[i] = from->end[i]; @@ -352,8 +359,10 @@ onig_region_copy(OnigRegion* to, OnigRegion* from) unsigned int size = (unsigned int )(((str_len) + 1) * (state_num) + 7) >> 3;\ offset = ((offset) * (state_num)) >> 3;\ if (size > 0 && offset < size && size < STATE_CHECK_BUFF_MAX_SIZE) {\ - if (size >= STATE_CHECK_BUFF_MALLOC_THRESHOLD_SIZE) \ + if (size >= STATE_CHECK_BUFF_MALLOC_THRESHOLD_SIZE) {\ (msa).state_check_buff = (void* )xmalloc(size);\ + CHECK_NULL_RETURN_MEMERR((msa).state_check_buff);\ + }\ else \ (msa).state_check_buff = (void* )xalloca(size);\ xmemset(((char* )((msa).state_check_buff)+(offset)), 0, \ @@ -378,7 +387,6 @@ onig_region_copy(OnigRegion* to, OnigRegion* from) }\ } while(0) #else -#define STATE_CHECK_BUFF_INIT(msa, str_len, offset, state_num) #define MATCH_ARG_FREE(msa) if ((msa).stack_p) xfree((msa).stack_p) #endif diff --git a/version.h b/version.h index c7bdbeb663..858d8b614f 100644 --- a/version.h +++ b/version.h @@ -1,7 +1,7 @@ #define RUBY_VERSION "1.9.0" -#define RUBY_RELEASE_DATE "2008-05-16" +#define RUBY_RELEASE_DATE "2008-05-17" #define RUBY_VERSION_CODE 190 -#define RUBY_RELEASE_CODE 20080516 +#define RUBY_RELEASE_CODE 20080517 #define RUBY_PATCHLEVEL 0 #define RUBY_VERSION_MAJOR 1 @@ -9,7 +9,7 @@ #define RUBY_VERSION_TEENY 0 #define RUBY_RELEASE_YEAR 2008 #define RUBY_RELEASE_MONTH 5 -#define RUBY_RELEASE_DAY 16 +#define RUBY_RELEASE_DAY 17 #ifdef RUBY_EXTERN RUBY_EXTERN const char ruby_version[];