diff options
| author | h2so5 <[email protected]> | 2013-08-24 06:49:58 +0900 |
|---|---|---|
| committer | h2so5 <[email protected]> | 2013-08-24 06:52:46 +0900 |
| commit | 23df879a81161a0b87721f422ca7e68f63f6746e (patch) | |
| tree | 75e0d14f451b8b4ccbdda6f2b057a01ae701f4a3 /src | |
| parent | aa099f7f7d1f8c61db936fb7c39a3c734175af16 (diff) | |
| download | mruby-23df879a81161a0b87721f422ca7e68f63f6746e.tar.gz mruby-23df879a81161a0b87721f422ca7e68f63f6746e.zip | |
Fix a problem of 'z' option and shared string
Diffstat (limited to 'src')
| -rw-r--r-- | src/class.c | 7 | ||||
| -rw-r--r-- | src/string.c | 1 |
2 files changed, 7 insertions, 1 deletions
diff --git a/src/class.c b/src/class.c index b5c561d06..df034737e 100644 --- a/src/class.c +++ b/src/class.c @@ -489,14 +489,19 @@ mrb_get_args(mrb_state *mrb, const char *format, ...) mrb_value ss; struct RString *s; char **ps; + mrb_int len; ps = va_arg(ap, char**); if (i < argc) { ss = to_str(mrb, *sp++); s = mrb_str_ptr(ss); - if ((mrb_int)strlen(s->ptr) < s->len) { + len = (mrb_int)strlen(s->ptr); + if (len < s->len) { mrb_raise(mrb, E_ARGUMENT_ERROR, "String contains NUL"); } + else if (len > s->len) { + mrb_str_modify(mrb, s); + } *ps = s->ptr; i++; } diff --git a/src/string.c b/src/string.c index 2004c9fe8..88dcfc63e 100644 --- a/src/string.c +++ b/src/string.c @@ -62,6 +62,7 @@ mrb_str_modify(mrb_state *mrb, struct RString *s) if (shared->refcnt == 1 && s->ptr == shared->ptr) { s->ptr = shared->ptr; s->aux.capa = shared->len; + s->ptr[s->len] = '\0'; mrb_free(mrb, shared); } else { |
