diff options
author | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2016-05-19 03:10:12 +0000 |
---|---|---|
committer | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2016-05-19 03:10:12 +0000 |
commit | a28c12af1458f25f3c0d9c0b24908fcee1a44ad3 (patch) | |
tree | 77998ac89b0fad3626652a2bd77dd0ca2be06cf4 /re.c | |
parent | 0c221f631cc98b3194edcb7c4f4bcb536be0c3fa (diff) |
re.c: fix match?
* re.c (rb_reg_match_m_p): fix match against empty string.
rb_str_offset returns the end when the position exceeds the
length. fix the range parameter of onig_search.
[ruby-core:75604] [Bug #12394]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@55069 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 're.c')
-rw-r--r-- | re.c | 23 |
1 files changed, 10 insertions, 13 deletions
@@ -3227,6 +3227,7 @@ rb_reg_match_m_p(int argc, VALUE *argv, VALUE re) regex_t *reg; onig_errmsg_buffer err = ""; OnigPosition result; + const UChar *start, *end; int tmpreg; rb_scan_args(argc, argv, "11", &str, &initpos); @@ -3234,27 +3235,23 @@ rb_reg_match_m_p(int argc, VALUE *argv, VALUE re) str = SYMBOL_P(str) ? rb_sym2str(str) : rb_str_to_str(str); if (argc == 2) { pos = NUM2LONG(initpos); - if (pos == 0) goto run; if (pos < 0) { pos += NUM2LONG(rb_str_length(str)); - if (pos == 0) goto run; if (pos < 0) return Qfalse; - } - pos = rb_str_offset(str, pos); - } -run: - if (pos >= RSTRING_LEN(str)) { - return Qfalse; + if (pos > 0) { + long len = 0; + char *beg = rb_str_subpos(str, pos, &len); + if (!beg) return Qfalse; + pos = beg - RSTRING_PTR(str); + } } reg = rb_reg_prepare_re0(re, str, err); tmpreg = reg != RREGEXP_PTR(re); if (!tmpreg) RREGEXP(re)->usecnt++; - result = onig_search(reg, - ((UChar*)RSTRING_PTR(str)), - ((UChar*)RSTRING_END(str)), - ((UChar*)(RSTRING_PTR(str)) + pos), - ((UChar*)RSTRING_PTR(str)), + start = ((UChar*)RSTRING_PTR(str)); + end = start + RSTRING_LEN(str); + result = onig_search(reg, start, end, start + pos, end, NULL, ONIG_OPTION_NONE); if (!tmpreg) RREGEXP(re)->usecnt--; if (tmpreg) { |