Skip Menu |

This queue is for tickets about the JSON CPAN distribution.

Report information
The Basics
Id: 20066
Status: resolved
Priority: 0/
Queue: JSON

People
Owner: Nobody in particular
Requestors: coadysean [...] hotmail.com
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: 1.07
Fixed in: (no value)



Subject: '/' should be escaped in Converter.pm
Browsers parse the </script> tag even if it appears inside a string literal in javascript. In other words, the following HTML: <script> document.write("</script>"); </script> is invalid because the </script> inside the document.write() will be interpreted as the end of the javascript. The simple fix is to always escape the '/' character in Converter.pm: my %esc = ( "\n" => '\n', "\r" => '\r', "\t" => '\t', "\f" => '\f', "\b" => '\b', "\"" => '\"', "\\" => '\\\\', "\'" => '\\\'', "/" => '\\/', ); sub _stringfy { my ($arg) = @_; $arg =~ s/([\/\\"\n\r\t\f\b])/$esc{$1}/eg; $arg =~ s/([\x00-\x07\x0b\x0e-\x1f])/'\\u00' . unpack('H2',$1)/eg; $JSON::Converter::utf8 and utf8::decode($arg); return '"' . $arg . '"'; } sub _stringfy_single_quote { my $arg = shift; $arg =~ s/([\/\\\n'\r\t\f\b])/$esc{$1}/eg; $arg =~ s/([\x00-\x07\x0b\x0e-\x1f])/'\\u00' . unpack('H2',$1)/eg; $JSON::Converter::utf8 and utf8::decode($arg); return "'" . $arg ."'"; }; ====== This was tested using Perl 5.8.5 Thanks Sean Coady
Subject: Converter.pm
package JSON::Converter; ############################################################################## use Carp; use vars qw($VERSION $USE_UTF8); use strict; use JSON (); $VERSION = '1.09'; BEGIN { eval 'require Scalar::Util'; unless($@){ *JSON::Converter::blessed = \&Scalar::Util::blessed; } else{ # This code is from Sclar::Util. # warn $@; eval 'sub UNIVERSAL::a_sub_not_likely_to_be_here { ref($_[0]) }'; *JSON::Converter::blessed = sub { local($@, $SIG{__DIE__}, $SIG{__WARN__}); ref($_[0]) ? eval { $_[0]->a_sub_not_likely_to_be_here } : undef; }; } $USE_UTF8 = JSON->USE_UTF8; } ############################################################################## sub new { my $class = shift; bless {indent => 2, pretty => 0, delimiter => 2, @_}, $class; } sub objToJson { my $self = shift; my $obj = shift; my $opt = shift; local(@{$self}{qw/autoconv execcoderef skipinvalid/}); local(@{$self}{qw/pretty indent delimiter keysort convblessed utf8 singlequote/}); $self->_initConvert($opt); if($self->{convblessed}){ $obj = _blessedToNormalObject($obj); } #(not hash for speed) local @JSON::Converter::obj_addr; # check circular references # for speed local $JSON::Converter::pretty = $self->{pretty}; local $JSON::Converter::keysort = !$self->{keysort} ? undef : ref($self->{keysort}) eq 'CODE' ? $self->{keysort} : $self->{keysort} =~ /\D+/ ? $self->{keysort} : sub { $a cmp $b }; local $JSON::Converter::autoconv = $self->{autoconv}; local $JSON::Converter::execcoderef = $self->{execcoderef}; local $JSON::Converter::selfconvert = $self->{selfconvert}; local $JSON::Converter::utf8 = $self->{utf8}; local *_stringfy = *_stringfy_single_quote if($self->{singlequote}); return $self->_toJson($obj); } *hashToJson = \&objToJson; *arrayToJson = \&objToJson; *valueToJson = \&_valueToJson; sub _toJson { my ($self, $obj) = @_; if(ref($obj) eq 'HASH'){ return $self->_hashToJson($obj); } elsif(ref($obj) eq 'ARRAY'){ return $self->_arrayToJson($obj); } elsif( $JSON::Converter::selfconvert and blessed($obj) and $obj->can('toJson') ){ return $self->_selfToJson($obj); } else{ return; } } sub _hashToJson { my ($self, $obj) = @_; my ($k,$v); my %res; my ($pre,$post) = $self->_upIndent() if($JSON::Converter::pretty); if(grep { $_ == $obj } @JSON::Converter::obj_addr){ die "circle ref!"; } push @JSON::Converter::obj_addr,$obj; for my $k (keys %$obj){ my $v = $obj->{$k}; $res{$k} = $self->_toJson($v) || $self->_valueToJson($v); } pop @JSON::Converter::obj_addr; if($JSON::Converter::pretty){ $self->_downIndent(); my $del = $self->{_delstr}; return "{$pre" . join(",$pre", map { _stringfy($_) . $del .$res{$_} } (defined $JSON::Converter::keysort ? ( sort $JSON::Converter::keysort (keys %res)) : (keys %res) ) ). "$post}"; } else{ return '{'. join(',',map { _stringfy($_) .':' .$res{$_} } (defined $JSON::Converter::keysort ? ( sort $JSON::Converter::keysort (keys %res)) : (keys %res) ) ) .'}'; } } sub _arrayToJson { my ($self, $obj) = @_; my @res; my ($pre,$post) = $self->_upIndent() if($JSON::Converter::pretty); if(grep { $_ == $obj } @JSON::Converter::obj_addr){ die "circle ref!"; } push @JSON::Converter::obj_addr,$obj; for my $v (@$obj){ push @res, $self->_toJson($v) || $self->_valueToJson($v); } pop @JSON::Converter::obj_addr; if($JSON::Converter::pretty){ $self->_downIndent(); return "[$pre" . join(",$pre" ,@res) . "$post]"; } else{ return '[' . join(',' ,@res) . ']'; } } sub _selfToJson { my ($self, $obj) = @_; if(grep { $_ == $obj } @JSON::Converter::obj_addr){ die "circle ref!"; } push @JSON::Converter::obj_addr, $obj; return $obj->toJson($self); } sub _valueToJson { my ($self, $value) = @_; return 'null' if(!defined $value); if(!ref($value)){ if($JSON::Converter::autoconv){ return $value if($value =~ /^-?(?:0|[1-9][\d]*)(?:\.\d*)?(?:[eE][-+]?\d+)?$/); # return $value if($value =~ /^-?(?:0|[1-9][\d]*)(?:\.[\d]*)?$/); # return $value if($value =~ /^-?(?:\d+)(?:\.\d*)?(?:[eE][-+]?\d+)?$/); return $value if($value =~ /^0[xX](?:[0-9a-zA-Z])+$/); return 'true' if($value =~ /^[Tt][Rr][Uu][Ee]$/); return 'false' if($value =~ /^[Ff][Aa][Ll][Ss][Ee]$/); } return _stringfy($value); } elsif($JSON::Converter::execcoderef and ref($value) eq 'CODE'){ my $ret = $value->(); return 'null' if(!defined $ret); return $self->_toJson($ret) || _stringfy($ret); } elsif( blessed($value) and $value->isa('JSON::NotString') ){ return defined $value->{value} ? $value->{value} : 'null'; } else{ die "Invalid value" unless($self->{skipinvalid}); return 'null'; } } my %esc = ( "\n" => '\n', "\r" => '\r', "\t" => '\t', "\f" => '\f', "\b" => '\b', "\"" => '\"', "\\" => '\\\\', "\'" => '\\\'', "/" => '\\/', ); sub _stringfy { my ($arg) = @_; $arg =~ s/([\/\\"\n\r\t\f\b])/$esc{$1}/eg; $arg =~ s/([\x00-\x07\x0b\x0e-\x1f])/'\\u00' . unpack('H2',$1)/eg; $JSON::Converter::utf8 and utf8::decode($arg); return '"' . $arg . '"'; } sub _stringfy_single_quote { my $arg = shift; $arg =~ s/([\/\\\n'\r\t\f\b])/$esc{$1}/eg; $arg =~ s/([\x00-\x07\x0b\x0e-\x1f])/'\\u00' . unpack('H2',$1)/eg; $JSON::Converter::utf8 and utf8::decode($arg); return "'" . $arg ."'"; }; ############################################################################## sub _initConvert { my $self = shift; my %opt = %{ $_[0] } if(@_ > 0 and ref($_[0]) eq 'HASH'); $self->{autoconv} = $JSON::AUTOCONVERT if(!defined $self->{autoconv}); $self->{execcoderef} = $JSON::ExecCoderef if(!defined $self->{execcoderef}); $self->{skipinvalid} = $JSON::SkipInvalid if(!defined $self->{skipinvalid}); $self->{pretty} = $JSON::Pretty if(!defined $self->{pretty}); $self->{indent} = $JSON::Indent if(!defined $self->{indent}); $self->{delimiter} = $JSON::Delimiter if(!defined $self->{delimiter}); $self->{keysort} = $JSON::KeySort if(!defined $self->{keysort}); $self->{convblessed} = $JSON::ConvBlessed if(!defined $self->{convblessed}); $self->{selfconvert} = $JSON::SelfConvert if(!defined $self->{selfconvert}); $self->{utf8} = $JSON::UTF8 if(!defined $self->{utf8}); $self->{singlequote} = $JSON::SingleQuote if(!defined $self->{singlequote}); for my $name (qw/autoconv execcoderef skipinvalid pretty indent delimiter keysort convblessed selfconvert utf8 singlequote/){ $self->{$name} = $opt{$name} if(defined $opt{$name}); } if($self->{utf8} and !$USE_UTF8){ $self->{utf8} = 0; warn "JSON::Converter couldn't use utf8."; } $self->{indent_count} = 0; $self->{_delstr} = $self->{delimiter} ? ($self->{delimiter} == 1 ? ': ' : ' : ') : ':'; $self; } sub _upIndent { my $self = shift; my $space = ' ' x $self->{indent}; my ($pre,$post) = ('',''); $post = "\n" . $space x $self->{indent_count}; $self->{indent_count}++; $pre = "\n" . $space x $self->{indent_count}; return ($pre,$post); } sub _downIndent { $_[0]->{indent_count}--; } # # converting the blessed object to the normal object # sub _blessedToNormalObject { require overload; my ($obj) = @_; local @JSON::Converter::_blessedToNormal::obj_addr; return _blessedToNormal($obj); } sub _getObjType { return '' if(!ref($_[0])); ref($_[0]) eq 'HASH' ? 'HASH' : ref($_[0]) eq 'ARRAY' ? 'ARRAY' : $_[0]->isa('JSON::NotString') ? '' : (overload::StrVal($_[0]) =~ /=(\w+)/)[0]; } sub _blessedToNormal { my $type = _getObjType($_[0]); return $type eq 'HASH' ? _blessedToNormalHash($_[0]) : $type eq 'ARRAY' ? _blessedToNormalArray($_[0]) : $_[0]; } sub _blessedToNormalHash { my ($obj) = @_; my %res; die "circle ref!" if(grep { overload::AddrRef($_) eq overload::AddrRef($obj) } @JSON::Converter::_blessedToNormal::obj_addr); push @JSON::Converter::_blessedToNormal::obj_addr, $obj; for my $k (keys %$obj){ $res{$k} = _blessedToNormal($obj->{$k}); } pop @JSON::Converter::_blessedToNormal::obj_addr; return \%res; } sub _blessedToNormalArray { my ($obj) = @_; my @res; die "circle ref!" if(grep { overload::AddrRef($_) eq overload::AddrRef($obj) } @JSON::Converter::_blessedToNormal::obj_addr); push @JSON::Converter::_blessedToNormal::obj_addr, $obj; for my $v (@$obj){ push @res, _blessedToNormal($v); } pop @JSON::Converter::_blessedToNormal::obj_addr; return \@res; } ############################################################################## 1; __END__ =head1 METHODs =over =item objToJson convert a passed perl data structure into JSON object. can't parse bleesed object by default. =item hashToJson convert a passed hash into JSON object. =item arrayToJson convert a passed array into JSON array. =item valueToJson convert a passed data into a string of JSON. =back =head1 COPYRIGHT makamaka [at] donzoko.net This library is free software; you can redistribute it and/or modify it under the same terms as Perl itself. =head1 SEE ALSO L<JSON>, L<http://www.crockford.com/JSON/index.html> =cut
Since JSON-1.14, JSON::PP has escape_slash method. Please try JSON::PP.