diff options
-rw-r--r-- | ChangeLog | 5 | ||||
-rw-r--r-- | string.c | 4 |
2 files changed, 8 insertions, 1 deletions
@@ -1,3 +1,8 @@ +Wed Apr 14 15:58:11 2010 Nobuyoshi Nakada <nobu@ruby-lang.org> + + * string.c (rb_string_value_cstr): make NUL terminated if it is + not done. + Wed Apr 14 12:56:21 2010 Shugo Maeda <shugo@ruby-lang.org> * lib/net/imap (encode_utf7): encode & properly. Thanks, Kengo @@ -1375,10 +1375,12 @@ rb_string_value_cstr(volatile VALUE *ptr) { VALUE str = rb_string_value(ptr); char *s = RSTRING_PTR(str); + long len = RSTRING_LEN(str); - if (!s || RSTRING_LEN(str) != (long)strlen(s)) { + if (!s || memchr(s, 0, len)) { rb_raise(rb_eArgError, "string contains null byte"); } + if (s[len]) rb_str_modify(str); return s; } |