From 368d9446213cdffd336c186a821f94ce49f725c1 Mon Sep 17 00:00:00 2001 From: mame Date: Wed, 30 Jul 2008 17:22:16 +0000 Subject: [PATCH] * ext/syck/rubyext.c (rb_syck_compile): remove meaningless branch that misleads into thinking that the variable sav may be uninitialized. git-svn-id: svn+ssh://ci.ruby-lang.org/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 809d87f0b0..2d2e84698c 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 bdf3c8c79c..cb3ad260ec 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';