From 6a158e05c880d420ddc66d991816b21026bc9f6f Mon Sep 17 00:00:00 2001 From: tadf Date: Tue, 31 May 2011 14:53:52 +0000 Subject: [PATCH] * ext/date/date_core.c (offset_to_sec): fixed invalid validation. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@31865 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- ChangeLog | 4 ++++ ext/date/date_core.c | 30 ++++++++++++++++++------------ 2 files changed, 22 insertions(+), 12 deletions(-) diff --git a/ChangeLog b/ChangeLog index 98b6ee8aaf..83245ed174 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,7 @@ +Tue May 31 23:49:08 2011 Tadayoshi Funaba + + * ext/date/date_core.c (offset_to_sec): fixed invalid validation. + Tue May 31 23:43:00 2011 Kenta Murata * ext/bigdecimal/bigdecimal.c (GetVpValueWithPrec): replace the algorithm for diff --git a/ext/date/date_core.c b/ext/date/date_core.c index bff8029cbe..d01086e7ea 100644 --- a/ext/date/date_core.c +++ b/ext/date/date_core.c @@ -1868,10 +1868,12 @@ offset_to_sec(VALUE vof, int *rof) { double n; - n = NUM2DBL(vof); + n = NUM2DBL(vof) * DAY_IN_SECONDS; if (n < -DAY_IN_SECONDS || n > DAY_IN_SECONDS) return 0; - *rof = round(n * DAY_IN_SECONDS); + *rof = round(n); + if (*rof != n) + rb_warning("fraction of offset is ignored"); return 1; } case T_RATIONAL: @@ -1879,17 +1881,21 @@ offset_to_sec(VALUE vof, int *rof) VALUE vs = day_to_sec(vof); VALUE vn = RRATIONAL(vs)->num; VALUE vd = RRATIONAL(vs)->den; - long n, d; + long n; - if (!FIXNUM_P(vn) || !FIXNUM_P(vd)) - return 0; - n = FIX2LONG(vn); - d = FIX2LONG(vd); - if (d != 1) - return 0; - if (n < -DAY_IN_SECONDS || n > DAY_IN_SECONDS) - return 0; - *rof = n; + if (FIXNUM_P(vn) && FIXNUM_P(vd) && (FIX2LONG(vd) == 1)) + n = FIX2LONG(vn); + else { + vn = f_round(vs); + if (!f_eqeq_p(vn, vs)) + rb_warning("fraction of offset is ignored"); + if (!FIXNUM_P(vn)) + return 0; + n = FIX2LONG(vn); + if (n < -DAY_IN_SECONDS || n > DAY_IN_SECONDS) + return 0; + } + *rof = (int)n; return 1; } case T_STRING: