summaryrefslogtreecommitdiff
path: root/dir.c
diff options
context:
space:
mode:
authornagachika <nagachika@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>2013-06-04 15:09:53 +0000
committernagachika <nagachika@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>2013-06-04 15:09:53 +0000
commit0c91b2493c566b407e1ce9a64ed1a367f353716f (patch)
treeb5cc03d64f470f30678286a5f40f32abe56994be /dir.c
parent28b7866304e66f3f77b5515adc508bb8cb7283d0 (diff)
merge revision(s) 40925: [Backport #8451]
* dir.c (bracket): fix copy-paste error. When the first and last characters of fnmatch range have different length, fnmatch may have wrongly matched a path that does not really match. Coverity Scan found this bug. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_2_0_0@41068 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'dir.c')
-rw-r--r--dir.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/dir.c b/dir.c
index a46906e1af..17192141e7 100644
--- a/dir.c
+++ b/dir.c
@@ -137,7 +137,7 @@ bracket(
p = t2 + (r2 = rb_enc_mbclen(t2, pend, enc));
if (ok) continue;
if ((r <= (send-s) && memcmp(t1, s, r) == 0) ||
- (r2 <= (send-s) && memcmp(t2, s, r) == 0)) {
+ (r2 <= (send-s) && memcmp(t2, s, r2) == 0)) {
ok = 1;
continue;
}