Re: YAML::Syck::Loader#transfer dumps core

Moin,

At Fri, 24 Dec 2004 00:00:00 +0900,
Tanaka Akira wrote in [ruby-core:04010]:
> 
> Ruby dumps core as follows.
> 
> % ./ruby -ryaml -e '
> YAML::Syck::Loader.new.transfer(0, 0)'
> -e:2: [BUG] Segmentation fault
> ruby 1.9.0 (2004-12-23) [i686-linux]
> 
> zsh: abort      ./ruby -ryaml -e ' YAML::Syck::Loader.new.transfer(0, 0)'

A patch.


Index: ext/syck/rubyext.c
===================================================================
RCS file: /cvs/ruby/src/ruby/ext/syck/rubyext.c,v
retrieving revision 1.44
diff -U2 -p -w -r1.44 rubyext.c
--- ext/syck/rubyext.c	15 Dec 2004 16:01:10 -0000	1.44
+++ ext/syck/rubyext.c	23 Dec 2004 16:15:06 -0000
@@ -1069,5 +1069,5 @@ syck_loader_transfer( self, type, val )
     char *taguri = NULL;
 
-    if (NIL_P(type) || !RSTRING(type)->ptr || RSTRING(type)->len == 0) 
+    if (NIL_P(type) || RSTRING(StringValue(type))->len == 0)
     {
         /*


-- 
Nobu Nakada