diff --git a/ChangeLog b/ChangeLog index 241a18d695..992e6e7c19 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,9 @@ +Wed Oct 15 22:19:14 2008 NAKAMURA Usaku + + * array.c (RESIZE_CAPA): check whether len is longer than capacity. + + * array.c (rb_ary_compact_bang): resize ary before changing capacity. + Wed Oct 15 16:57:30 2008 NAKAMURA Usaku * version.h (RUBY_DESCRIPTION): remove unnecessary space. diff --git a/array.c b/array.c index 4aa6dbef07..da513f96aa 100644 --- a/array.c +++ b/array.c @@ -125,6 +125,7 @@ memfill(register VALUE *mem, register long size, register VALUE val) static void RESIZE_CAPA(VALUE ary, long capacity) { + assert(RARRAY_LEN(ary) <= capacity); assert(!OBJ_FROZEN(ary)); assert(!ARY_SHARED_P(ary)); if (capacity > RARRAY_EMBED_LEN_MAX) { @@ -2927,10 +2928,10 @@ rb_ary_compact_bang(VALUE ary) if (RARRAY_LEN(ary) == n) { return Qnil; } + ARY_SET_LEN(ary, n); if (n * 2 < ARY_CAPA(ary) && ARY_DEFAULT_SIZE * 2 < ARY_CAPA(ary)) { RESIZE_CAPA(ary, n * 2); } - ARY_SET_LEN(ary, n); return ary; }