From 9cc62abc6b0245e5f0e4a8f0cd9b6f53a4f9bf77 Mon Sep 17 00:00:00 2001 From: nobu Date: Sat, 5 Feb 2011 01:30:01 +0000 Subject: * ext/json/parser/parser.h (GET_PARSER): check if initialized. [ruby-core:35079] * ext/json/parser/parser.rl (cParser_initialize): ditto. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@30791 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- ext/json/parser/parser.c | 17 +++++++++++------ ext/json/parser/parser.h | 3 +++ ext/json/parser/parser.rl | 7 ++++++- 3 files changed, 20 insertions(+), 7 deletions(-) (limited to 'ext/json/parser') diff --git a/ext/json/parser/parser.c b/ext/json/parser/parser.c index b4faa864d8..22c8167fc7 100644 --- a/ext/json/parser/parser.c +++ b/ext/json/parser/parser.c @@ -1610,7 +1610,11 @@ static VALUE cParser_initialize(int argc, VALUE *argv, VALUE self) char *ptr; long len; VALUE source, opts; - GET_PARSER; + GET_PARSER_INIT; + + if (json->Vsource) { + rb_raise(rb_eArgError, "already initialized instance"); + } rb_scan_args(argc, argv, "11", &source, &opts); source = convert_encoding(StringValue(source)); ptr = RSTRING_PTR(source); @@ -1698,16 +1702,16 @@ static VALUE cParser_parse(VALUE self) GET_PARSER; -#line 1702 "parser.c" +#line 1706 "parser.c" { cs = JSON_start; } -#line 699 "parser.rl" +#line 703 "parser.rl" p = json->source; pe = p + json->len; -#line 1711 "parser.c" +#line 1715 "parser.c" { if ( p == pe ) goto _test_eof; @@ -1784,7 +1788,7 @@ st10: if ( ++p == pe ) goto _test_eof10; case 10: -#line 1788 "parser.c" +#line 1792 "parser.c" switch( (*p) ) { case 13: goto st10; case 32: goto st10; @@ -1841,7 +1845,7 @@ case 9: _out: {} } -#line 702 "parser.rl" +#line 706 "parser.rl" if (cs >= JSON_first_final && p == pe) { return result; @@ -1938,5 +1942,6 @@ void Init_parser() * Local variables: * mode: c * c-file-style: ruby + * indent-tabs-mode: nil * End: */ diff --git a/ext/json/parser/parser.h b/ext/json/parser/parser.h index 688ffdaeba..5d4532b948 100644 --- a/ext/json/parser/parser.h +++ b/ext/json/parser/parser.h @@ -44,6 +44,9 @@ typedef struct JSON_ParserStruct { } JSON_Parser; #define GET_PARSER \ + GET_PARSER_INIT; \ + if (!json->Vsource) rb_raise(rb_eArgError, "uninitialized instance") +#define GET_PARSER_INIT \ JSON_Parser *json; \ Data_Get_Struct(self, JSON_Parser, json) diff --git a/ext/json/parser/parser.rl b/ext/json/parser/parser.rl index 3ec7d1ce14..680242732e 100644 --- a/ext/json/parser/parser.rl +++ b/ext/json/parser/parser.rl @@ -608,7 +608,11 @@ static VALUE cParser_initialize(int argc, VALUE *argv, VALUE self) char *ptr; long len; VALUE source, opts; - GET_PARSER; + GET_PARSER_INIT; + + if (json->Vsource) { + rb_raise(rb_eArgError, "already initialized instance"); + } rb_scan_args(argc, argv, "11", &source, &opts); source = convert_encoding(StringValue(source)); ptr = RSTRING_PTR(source); @@ -795,5 +799,6 @@ void Init_parser() * Local variables: * mode: c * c-file-style: ruby + * indent-tabs-mode: nil * End: */ -- cgit v1.2.3