diff options
author | mame <mame@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2008-12-22 15:18:12 +0000 |
---|---|---|
committer | mame <mame@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2008-12-22 15:18:12 +0000 |
commit | 41da4d16b0bbfa12827ac772c28bf43b0b655140 (patch) | |
tree | a12e4d262e3a4bacec7f32ab34aa283b58847658 | |
parent | 8f79efe0becb0ca996bc9affe2476b616f91fcde (diff) | |
download | ruby-41da4d16b0bbfa12827ac772c28bf43b0b655140.tar.gz |
* sprintf.c (rb_str_format): fix buffer overflow.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@20921 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 4 | ||||
-rw-r--r-- | sprintf.c | 4 |
2 files changed, 6 insertions, 2 deletions
@@ -1,3 +1,7 @@ +Tue Dec 23 00:16:48 2008 Yusuke Endoh <mame@tsg.ne.jp> + + * sprintf.c (rb_str_format): fix buffer overflow. + Mon Dec 22 19:31:19 2008 Yuki Sonoda (Yugui) <yugui@yugui.jp> * common.mk (revision.h): uses tool/file2lastrev.rb to support @@ -979,8 +979,8 @@ rb_str_format(int argc, const VALUE *argv, VALUE fmt) if ((flags & FWIDTH) && need < width) need = width; - CHECK(need); - snprintf(&buf[blen], need, "%*s", need, ""); + CHECK(need + 1); + snprintf(&buf[blen], need + 1, "%*s", need, ""); if (flags & FMINUS) { if (!isnan(fval) && fval < 0.0) buf[blen++] = '-'; |