summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--ChangeLog4
-rw-r--r--time.c22
2 files changed, 25 insertions, 1 deletions
diff --git a/ChangeLog b/ChangeLog
index 6da3db1dc9..47d05f8ce4 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+Thu Jun 16 14:32:31 2011 NARUSE, Yui <naruse@ruby-lang.org>
+
+ * time.c (rb_time_new): prevent overflow by "* 1000".
+
Fri Jun 17 03:07:36 2011 Koichi Sasada <ko1@atdot.net>
* benchmark/bm_vm4_thread_create_join.rb,
diff --git a/time.c b/time.c
index d2c5ecc163..20eea4728f 100644
--- a/time.c
+++ b/time.c
@@ -2299,7 +2299,27 @@ time_new_timew(VALUE klass, wideval_t timew)
VALUE
rb_time_new(time_t sec, long usec)
{
- return time_new_timew(rb_cTime, nsec2timew(sec, usec * 1000));
+ wideval_t timew;
+
+ if (usec >= 1000000) {
+ long sec2 = usec / 1000000;
+ if (sec > TIMET_MAX - sec2) {
+ rb_raise(rb_eRangeError, "out of Time range");
+ }
+ usec -= sec2 * 1000000;
+ sec += sec2;
+ }
+ else if (usec <= 1000000) {
+ long sec2 = usec / 1000000;
+ if (sec < -TIMET_MAX - sec2) {
+ rb_raise(rb_eRangeError, "out of Time range");
+ }
+ usec -= sec2 * 1000000;
+ sec += sec2;
+ }
+
+ timew = nsec2timew(sec, usec * 1000);
+ return time_new_timew(rb_cTime, timew);
}
VALUE