From 3ab3ec62c32a7791706ade4052e65286c9be2f99 Mon Sep 17 00:00:00 2001 From: mame Date: Wed, 30 Jul 2008 17:22:16 +0000 Subject: * ext/syck/rubyext.c (rb_syck_compile): remove meaningless branch that misleads into thinking that the variable sav may be uninitialized. git-svn-id: http://svn.ruby-lang.org/repos/ruby/trunk@18282 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- ChangeLog | 6 ++++++ ext/syck/rubyext.c | 5 +++-- 2 files changed, 9 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/ChangeLog index 809d87f0b..2d2e84698 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,9 @@ +Thu Jul 31 02:21:39 2008 Yusuke Endoh + + * ext/syck/rubyext.c (rb_syck_compile): remove meaningless branch + that misleads into thinking that the variable sav may be + uninitialized. + Thu Jul 31 01:25:43 2008 Nobuyoshi Nakada * thread.c (rb_mutex_unlock_all): mutex is no longer a ruby object. diff --git a/ext/syck/rubyext.c b/ext/syck/rubyext.c index bdf3c8c79..cb3ad260e 100644 --- a/ext/syck/rubyext.c +++ b/ext/syck/rubyext.c @@ -104,7 +104,7 @@ rb_syck_compile(VALUE self, VALUE port) char *ret; VALUE bc; bytestring_t *sav = NULL; - void *data; + void *data = NULL; SyckParser *parser = syck_new_parser(); taint = syck_parser_assign_io(parser, &port); @@ -113,7 +113,8 @@ rb_syck_compile(VALUE self, VALUE port) syck_parser_implicit_typing( parser, 0 ); syck_parser_taguri_expansion( parser, 0 ); oid = syck_parse( parser ); - if (syck_lookup_sym( parser, oid, &data )) sav = data; + ASSERT(syck_lookup_sym( parser, oid, &data )); + sav = data; ret = S_ALLOCA_N( char, strlen( sav->buffer ) + 3 ); ret[0] = '\0'; -- cgit