mirror of
https://github.com/ruby/ruby.git
synced 2022-11-09 12:17:21 -05:00
cgi/escape: Optimize CGI.unescapeHTML
* cgi/escape/escape.c: Optimize CGI.unescapeHTML performance by C ext for ASCII-compatible encodings. [Fix GH-1242] git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@53884 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
This commit is contained in:
parent
6b6b3689dd
commit
1b7109e11a
2 changed files with 134 additions and 0 deletions
|
@ -1,3 +1,8 @@
|
||||||
|
Sun Feb 21 13:56:57 2016 Nobuyoshi Nakada <nobu@ruby-lang.org>
|
||||||
|
|
||||||
|
* cgi/escape/escape.c: Optimize CGI.unescapeHTML performance by C
|
||||||
|
ext for ASCII-compatible encodings. [Fix GH-1242]
|
||||||
|
|
||||||
Sat Feb 20 15:38:16 2016 Eric Wong <e@80x24.org>
|
Sat Feb 20 15:38:16 2016 Eric Wong <e@80x24.org>
|
||||||
|
|
||||||
* doc/extension.rdoc: update paths for defs/ directory
|
* doc/extension.rdoc: update paths for defs/ directory
|
||||||
|
|
|
@ -1,6 +1,7 @@
|
||||||
#include "ruby.h"
|
#include "ruby.h"
|
||||||
#include "ruby/encoding.h"
|
#include "ruby/encoding.h"
|
||||||
|
|
||||||
|
RUBY_EXTERN unsigned long ruby_scan_digits(const char *str, ssize_t len, int base, size_t *retlen, int *overflow);
|
||||||
RUBY_EXTERN const char ruby_hexdigits[];
|
RUBY_EXTERN const char ruby_hexdigits[];
|
||||||
#define lower_hexdigits (ruby_hexdigits+0)
|
#define lower_hexdigits (ruby_hexdigits+0)
|
||||||
#define upper_hexdigits (ruby_hexdigits+16)
|
#define upper_hexdigits (ruby_hexdigits+16)
|
||||||
|
@ -76,6 +77,113 @@ optimized_escape_html(VALUE str)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
static VALUE
|
||||||
|
optimized_unescape_html(VALUE str)
|
||||||
|
{
|
||||||
|
enum {UNICODE_MAX = 0x10ffff};
|
||||||
|
rb_encoding *enc = rb_enc_get(str);
|
||||||
|
unsigned long charlimit = (strcasecmp(rb_enc_name(enc), "UTF-8") == 0 ? UNICODE_MAX :
|
||||||
|
strcasecmp(rb_enc_name(enc), "ISO-8859-1") == 0 ? 256 :
|
||||||
|
128);
|
||||||
|
long i, len, beg = 0;
|
||||||
|
size_t clen, plen;
|
||||||
|
int overflow;
|
||||||
|
const char *cstr;
|
||||||
|
char buf[6];
|
||||||
|
VALUE dest = 0;
|
||||||
|
|
||||||
|
len = RSTRING_LEN(str);
|
||||||
|
cstr = RSTRING_PTR(str);
|
||||||
|
|
||||||
|
for (i = 0; i < len; i++) {
|
||||||
|
unsigned long cc;
|
||||||
|
char c = cstr[i];
|
||||||
|
if (c != '&') continue;
|
||||||
|
plen = i - beg;
|
||||||
|
if (++i >= len) break;
|
||||||
|
c = (unsigned char)cstr[i];
|
||||||
|
switch (c) {
|
||||||
|
case 'a':
|
||||||
|
++i;
|
||||||
|
if (len - i >= 4 && memcmp(&cstr[i], "pos;", 4) == 0) {
|
||||||
|
c = '\'';
|
||||||
|
i += 3;
|
||||||
|
}
|
||||||
|
else if (len - i >= 3 && memcmp(&cstr[i], "mp;", 3) == 0) {
|
||||||
|
c = '&';
|
||||||
|
i += 2;
|
||||||
|
}
|
||||||
|
else continue;
|
||||||
|
break;
|
||||||
|
case 'q':
|
||||||
|
++i;
|
||||||
|
if (len - i >= 4 && memcmp(&cstr[i], "uot;", 4) == 0) {
|
||||||
|
c = '"';
|
||||||
|
i += 3;
|
||||||
|
}
|
||||||
|
else continue;
|
||||||
|
break;
|
||||||
|
case 'g':
|
||||||
|
++i;
|
||||||
|
if (len - i >= 2 && memcmp(&cstr[i], "t;", 2) == 0) {
|
||||||
|
c = '>';
|
||||||
|
i += 1;
|
||||||
|
}
|
||||||
|
else continue;
|
||||||
|
break;
|
||||||
|
case 'l':
|
||||||
|
++i;
|
||||||
|
if (len - i >= 2 && memcmp(&cstr[i], "t;", 2) == 0) {
|
||||||
|
c = '<';
|
||||||
|
i += 1;
|
||||||
|
}
|
||||||
|
else continue;
|
||||||
|
break;
|
||||||
|
case '#':
|
||||||
|
if (len - ++i >= 2 && ISDIGIT(cstr[i])) {
|
||||||
|
cc = ruby_scan_digits(&cstr[i], len-i, 10, &clen, &overflow);
|
||||||
|
}
|
||||||
|
else if ((cstr[i] == 'x' || cstr[i] == 'X') && len - ++i >= 2 && ISXDIGIT(cstr[i])) {
|
||||||
|
cc = ruby_scan_digits(&cstr[i], len-i, 16, &clen, &overflow);
|
||||||
|
}
|
||||||
|
else continue;
|
||||||
|
i += clen;
|
||||||
|
if (overflow || cc >= charlimit || cstr[i] != ';') continue;
|
||||||
|
if (!dest) {
|
||||||
|
dest = rb_str_buf_new(len);
|
||||||
|
}
|
||||||
|
rb_str_cat(dest, cstr + beg, plen);
|
||||||
|
if (charlimit > 256) {
|
||||||
|
rb_str_cat(dest, buf, rb_enc_mbcput((OnigCodePoint)cc, buf, enc));
|
||||||
|
}
|
||||||
|
else {
|
||||||
|
c = (unsigned char)cc;
|
||||||
|
rb_str_cat(dest, &c, 1);
|
||||||
|
}
|
||||||
|
beg = i + 1;
|
||||||
|
continue;
|
||||||
|
default:
|
||||||
|
--i;
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
if (!dest) {
|
||||||
|
dest = rb_str_buf_new(len);
|
||||||
|
}
|
||||||
|
rb_str_cat(dest, cstr + beg, plen);
|
||||||
|
rb_str_cat(dest, &c, 1);
|
||||||
|
beg = i + 1;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (dest) {
|
||||||
|
rb_str_cat(dest, cstr + beg, len - beg);
|
||||||
|
preserve_original_state(str, dest);
|
||||||
|
return dest;
|
||||||
|
}
|
||||||
|
else {
|
||||||
|
return rb_str_dup(str);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
static int
|
static int
|
||||||
url_unreserved_char(unsigned char c)
|
url_unreserved_char(unsigned char c)
|
||||||
{
|
{
|
||||||
|
@ -157,6 +265,26 @@ cgiesc_escape_html(VALUE self, VALUE str)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/*
|
||||||
|
* call-seq:
|
||||||
|
* CGI.unescapeHTML(string) -> string
|
||||||
|
*
|
||||||
|
* Returns HTML-unescaped string.
|
||||||
|
*
|
||||||
|
*/
|
||||||
|
static VALUE
|
||||||
|
cgiesc_unescape_html(VALUE self, VALUE str)
|
||||||
|
{
|
||||||
|
StringValue(str);
|
||||||
|
|
||||||
|
if (rb_enc_str_asciicompat_p(str)) {
|
||||||
|
return optimized_unescape_html(str);
|
||||||
|
}
|
||||||
|
else {
|
||||||
|
return rb_call_super(1, &str);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* call-seq:
|
* call-seq:
|
||||||
* CGI.escape(string) -> string
|
* CGI.escape(string) -> string
|
||||||
|
@ -184,6 +312,7 @@ Init_escape(void)
|
||||||
rb_mEscape = rb_define_module_under(rb_cCGI, "Escape");
|
rb_mEscape = rb_define_module_under(rb_cCGI, "Escape");
|
||||||
rb_mUtil = rb_define_module_under(rb_cCGI, "Util");
|
rb_mUtil = rb_define_module_under(rb_cCGI, "Util");
|
||||||
rb_define_method(rb_mEscape, "escapeHTML", cgiesc_escape_html, 1);
|
rb_define_method(rb_mEscape, "escapeHTML", cgiesc_escape_html, 1);
|
||||||
|
rb_define_method(rb_mEscape, "unescapeHTML", cgiesc_unescape_html, 1);
|
||||||
rb_define_method(rb_mEscape, "escape", cgiesc_escape, 1);
|
rb_define_method(rb_mEscape, "escape", cgiesc_escape, 1);
|
||||||
rb_prepend_module(rb_mUtil, rb_mEscape);
|
rb_prepend_module(rb_mUtil, rb_mEscape);
|
||||||
rb_extend_object(rb_cCGI, rb_mEscape);
|
rb_extend_object(rb_cCGI, rb_mEscape);
|
||||||
|
|
Loading…
Add table
Reference in a new issue