From ebe4711df82e70f5ac1be5921a5654f33746e185 Mon Sep 17 00:00:00 2001
From: cubicdaiya <cubicdaiya@gmail.com>
Date: Tue, 4 Mar 2014 21:58:47 +0900
Subject: fix off-by-one error in String#rindex(fixnum)

null-terminated string should not be included in search targets.
---
 src/string.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'src')

diff --git a/src/string.c b/src/string.c
index b7f45aef0..ad3ef2c70 100644
--- a/src/string.c
+++ b/src/string.c
@@ -1660,7 +1660,7 @@ mrb_str_rindex_m(mrb_state *mrb, mrb_value str)
       mrb_int len = RSTRING_LEN(str);
       unsigned char *p = (unsigned char*)RSTRING_PTR(str);
 
-      for (pos=len;pos>=0;pos--) {
+      for (pos=len-1;pos>=0;pos--) {
         if (p[pos] == c) return mrb_fixnum_value(pos);
       }
       return mrb_nil_value();
-- 
cgit v1.2.3