aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authornobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>2015-06-30 14:11:56 +0000
committernobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>2015-06-30 14:11:56 +0000
commit347da440cfefc7eec1d182110f4b42ac8e9af27b (patch)
tree3715717e1516f08d222326233db069e45c3cfdb7
parent24a72a177d3006482abe0bfe7d531d6575361679 (diff)
downloadruby-347da440cfefc7eec1d182110f4b42ac8e9af27b.tar.gz
ensure paths NUL-terminated
* dir.c (check_dirname): ensure path name NUL-terminated for SHARABLE_MIDDLE_SUBSTRING. * io.c (rb_sysopen): ditto. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@51073 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r--ChangeLog7
-rw-r--r--dir.c1
-rw-r--r--io.c1
3 files changed, 9 insertions, 0 deletions
diff --git a/ChangeLog b/ChangeLog
index 2e60021779..b5a1eeb2d5 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+Tue Jun 30 23:11:53 2015 Nobuyoshi Nakada <nobu@ruby-lang.org>
+
+ * dir.c (check_dirname): ensure path name NUL-terminated for
+ SHARABLE_MIDDLE_SUBSTRING.
+
+ * io.c (rb_sysopen): ditto.
+
Tue Jun 30 18:38:16 2015 NAKAMURA Usaku <usa@ruby-lang.org>
* win32/file.c (rb_freopen): need to terminate by NUL.
diff --git a/dir.c b/dir.c
index e271503dce..ee3032d69f 100644
--- a/dir.c
+++ b/dir.c
@@ -1019,6 +1019,7 @@ check_dirname(VALUE dir)
pend = rb_enc_path_end(rb_enc_path_skip_prefix(path, pend, enc), pend, enc);
if (pend - path < len) {
d = rb_str_subseq(d, 0, pend - path);
+ StringValueCStr(d);
}
return rb_str_encode_ospath(d);
}
diff --git a/io.c b/io.c
index 40d8920081..184f0e6496 100644
--- a/io.c
+++ b/io.c
@@ -5444,6 +5444,7 @@ rb_sysopen(VALUE fname, int oflags, mode_t perm)
struct sysopen_struct data;
data.fname = rb_str_encode_ospath(fname);
+ StringValueCStr(data.fname);
data.oflags = oflags;
data.perm = perm;