ruby-cvs:16171
From: (ocean) <ocean ruby-lang.org>
Date: Mon, 19 Dec 2005 11:42:35 +0900
Subject: [ruby-cvs:16171] ruby, ruby/ext/syck: * ext/syck/rubyext.c: sorry, I reverted my "should set newly
ocean Mon, 19 Dec 2005 11:42:35 +0900
Modified files:
ruby:
ChangeLog
ruby/ext/syck:
rubyext.c
Log:
* ext/syck/rubyext.c: sorry, I reverted my "should set newly
allocated memory instead of RString's internal storage" stuff.
node allocated in rubyext.c seems to be freed by rb_syck_free_node
not syck_free_node, and it won't free data.str->ptr and type_id.
(I still think this is unsafe because RString(foo)->ptr becomes
dangling pointer when RString is modified or freed, but anyway
I misunderstood, so go back to original code for now)
Revision Changes Path
1.4849 +11 -0 ruby/ChangeLog
http://www.ruby-lang.org/cgi-bin/cvsweb.cgi/ruby/ChangeLog?cvsroot=src&r1=1.4848&r2=1.4849
1.60 +3 -6 ruby/ext/syck/rubyext.c
http://www.ruby-lang.org/cgi-bin/cvsweb.cgi/ruby/ext/syck/rubyext.c?cvsroot=src&r1=1.59&r2=1.60