diff options
| -rw-r--r-- | ChangeLog | 4 | ||||
| -rw-r--r-- | ext/date/date_core.c | 30 |
2 files changed, 22 insertions, 12 deletions
@@ -1,3 +1,7 @@ +Tue May 31 23:49:08 2011 Tadayoshi Funaba <tadf@dotrb.org> + + * ext/date/date_core.c (offset_to_sec): fixed invalid validation. + Tue May 31 23:43:00 2011 Kenta Murata <mrkn@mrkn.jp> * 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: |