[ruby-dev:14313] Re: [BUG] segv on [str].pack("p")

まつもと ゆきひろです

In message "[ruby-dev:14299] Re: [BUG] segv on [str].pack("p")"
    on 01/08/06, [e-mail] <[e-mail]> writes:

|  やっぱりちゃんとrealloc()した方がいいですかね。associateして
|しまったStringを変更すること自体まずいので、capaはいらないと思
|うのですが。

capaは要りません。reallocした方が良いかも。

|  とりあえずpack.cの方だけでもcommitしておいていいですか? > ま
|つもとさん。

はい。string.cはrb_str_associateを前の仕様のままにする(つま
りassociatesではなく個別にrb_str_associateを呼ぶ)形式にした
方が良いんではないかと思います。

こんな感じ。

--- pack.c	2001/05/30 09:12:34	1.26
+++ pack.c	2001/08/06 02:30:22
@@ -333,3 +333,3 @@
     char *p, *pend;
-    VALUE res, from, associates = 0;
+    VALUE res, from;
     char type;
@@ -874,6 +874,3 @@
 		}
-		if (!associates) {
-		    associates = rb_ary_new();
-		}
-		rb_ary_push(associates, from);
+		rb_str_associate(res, from);
 		rb_str_buf_cat(res, (char*)&t, sizeof(char*));
@@ -931,6 +928,2 @@
 	}
-    }
-
-    if (associates) {
-	rb_str_associate(res, associates);
     }