From 7d4115921291c8691ce46061a924f08a9607b07b 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: svn+ssh://ci.ruby-lang.org/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 a3b9bb6394..b690701d17 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 3636006621..c17f34baaf 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 v1.2.3