[ruby-dev:12629] Re: Marshal bug still remains

えぐち＠エスアンドイー です。

>>> In message [ruby-dev:12621] Marshal bug still remains
    On Thu, 22 Mar 2001 22:30:17 +0900, "Akinori MUSHA" <[e-mail]> said:

> 　Marshal にまだバグが残っています。最新安定版で:
> 
> $ ruby -ve 'Marshal.load Marshal.dump(2**32)'
> ruby 1.6.3 (2001-03-22) [i386-freebsd5.0]
> -e:1:in `load': marshal data too short (ArgumentError)
> 	from -e:1
> 
> となります。
> 
> 　いろいろ試してみると、 2**31 だと大丈夫、 3**32 も大丈夫、
> (2**32+2**16) は大丈夫、(2**32+2**15) は駄目、という感じで、
> 表面的な観察からは何がなんだか。0 の連続の仕方によるのかな。

これで正解？

  % ./miniruby -e 'p (Marshal.load Marshal.dump(2**32))'
  4294967296

0 の扱いでビンゴ風．
あとインクリメント量はこちらが正解では？

	えぐち

Index: marshal.c
===================================================================
RCS file: /home/eguchi/cvs.ruby-lang.org/ruby/marshal.c,v
retrieving revision 1.32
diff -d -u -r1.32 marshal.c
--- marshal.c	2001/03/22 08:59:02	1.32
+++ marshal.c	2001/03/22 14:04:51
@@ -376,10 +376,9 @@
 		    BDIGIT num = *d;
 		    int i;
 
-		    for (i=0; i<SIZEOF_BDIGITS; i+=sizeof(short)) {
+		    for (i=0; i<SIZEOF_BDIGITS; i+=SIZEOF_BDIGITS / SIZEOF_SHORT) {
 			w_short(num & SHORTMASK, arg);
 			num = SHORTDN(num);
-			if (num == 0) break;
 		    }
 #else
 		    w_short(*d, arg);