From 109451716a482c0fe8e505223e5af0abb951f32b Mon Sep 17 00:00:00 2001 From: nobu Date: Fri, 9 Sep 2005 07:35:31 +0000 Subject: * string.c (rb_str_times): make empty strings to keep taintness, and a little improvement. [ruby-dev:26900] git-svn-id: http://svn.ruby-lang.org/repos/ruby/trunk@9104 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- ChangeLog | 5 +++++ string.c | 8 ++++---- 2 files changed, 9 insertions(+), 4 deletions(-) diff --git a/ChangeLog b/ChangeLog index a3b9bb639..b690701d1 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,8 @@ +Fri Sep 9 16:35:04 2005 Nobuyoshi Nakada + + * string.c (rb_str_times): make empty strings to keep taintness, + and a little improvement. [ruby-dev:26900] + Thu Sep 8 14:58:11 2005 Yukihiro Matsumoto * merged a patch from Takahiro Kambe to diff --git a/string.c b/string.c index 363600662..c17f34baa 100644 --- a/string.c +++ b/string.c @@ -412,13 +412,13 @@ rb_str_times(str, times) if (len < 0) { rb_raise(rb_eArgError, "negative argument"); } - if (LONG_MAX/len < RSTRING(str)->len) { + if (len && LONG_MAX/len < RSTRING(str)->len) { rb_raise(rb_eArgError, "argument too big"); } - str2 = rb_str_new5(str,0, RSTRING(str)->len*len); - for (i=0; iptr+(i*RSTRING(str)->len), + str2 = rb_str_new5(str,0, len *= RSTRING(str)->len); + for (i = 0; i < len; i += RSTRING(str)->len) { + memcpy(RSTRING(str2)->ptr + i, RSTRING(str)->ptr, RSTRING(str)->len); } RSTRING(str2)->ptr[RSTRING(str2)->len] = '\0'; -- cgit