diff options
author | matz <matz@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2004-10-02 03:49:15 +0000 |
---|---|---|
committer | matz <matz@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2004-10-02 03:49:15 +0000 |
commit | 9d923be60446cd599094d12f259b7fee173ec875 (patch) | |
tree | 883110b737f81f4ab45efada98a26110bed59f8a | |
parent | 7665754a5a6cb8903d4892e3d00e709d5de7f0b6 (diff) | |
download | ruby-9d923be60446cd599094d12f259b7fee173ec875.tar.gz |
* string.c (rb_str_sum): check was done with false pointer.
[ruby-dev:24383]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@6978 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 3 | ||||
-rw-r--r-- | string.c | 8 |
2 files changed, 7 insertions, 4 deletions
@@ -1,5 +1,8 @@ Sat Oct 2 00:42:20 2004 Yukihiro Matsumoto <matz@ruby-lang.org> + * string.c (rb_str_sum): check was done with false pointer. + [ruby-dev:24383] + * string.c (rb_str_sum): string may be altered. [ruby-dev:24381] Fri Oct 1 11:40:14 2004 Yukihiro Matsumoto <matz@ruby-lang.org> @@ -4405,7 +4405,7 @@ rb_str_sum(argc, argv, str) { VALUE vbits; int bits; - char *p, *pend; + char *ptr, *p, *pend; long len; if (rb_scan_args(argc, argv, "01", &vbits) == 0) { @@ -4413,7 +4413,7 @@ rb_str_sum(argc, argv, str) } else bits = NUM2INT(vbits); - p = RSTRING(str)->ptr; + ptr = p = RSTRING(str)->ptr; len = RSTRING(str)->len; pend = p + len; if (bits > sizeof(long)*CHAR_BIT) { @@ -4424,7 +4424,7 @@ rb_str_sum(argc, argv, str) mod = rb_funcall(mod, '-', 1, INT2FIX(1)); while (p < pend) { - str_mod_check(str, p, len); + str_mod_check(str, ptr, len); res = rb_funcall(res, '+', 1, INT2FIX((unsigned int)*p)); p++; } @@ -4439,7 +4439,7 @@ rb_str_sum(argc, argv, str) mod = -1; } while (p < pend) { - str_mod_check(str, p, len); + str_mod_check(str, ptr, len); res += (unsigned int)*p; p++; } |