Raise proper exception when month argument is not a name

https://bugs.ruby-lang.org/issues/17485#change-89871
This commit is contained in:
Nobuyoshi Nakada 2021-12-25 16:19:33 +09:00
Родитель 42eb9bf37a
Коммит 0867b638af
2 изменённых файлов: 7 добавлений и 10 удалений

Просмотреть файл

@ -49,7 +49,10 @@ class TestTime < Test::Unit::TestCase
t = Time.new(*tm, "-12:00") t = Time.new(*tm, "-12:00")
assert_equal([2001,2,28,23,59,30,-43200], [t.year, t.month, t.mday, t.hour, t.min, t.sec, t.gmt_offset], bug4090) assert_equal([2001,2,28,23,59,30,-43200], [t.year, t.month, t.mday, t.hour, t.min, t.sec, t.gmt_offset], bug4090)
assert_raise(ArgumentError) { Time.new(2000,1,1, 0,0,0, "+01:60") } assert_raise(ArgumentError) { Time.new(2000,1,1, 0,0,0, "+01:60") }
assert_raise(ArgumentError) { Time.new(2021, 1, 1, "+09:99") } msg = /invalid value for Integer/
assert_raise_with_message(ArgumentError, msg) { Time.new(2021, 1, 1, "+09:99") }
assert_raise_with_message(ArgumentError, msg) { Time.new(2021, 1, "+09:99") }
assert_raise_with_message(ArgumentError, msg) { Time.new(2021, "+09:99") }
end end
def test_time_add() def test_time_add()

12
time.c
Просмотреть файл

@ -2789,9 +2789,10 @@ month_arg(VALUE arg)
return obj2ubits(arg, 4); return obj2ubits(arg, 4);
} }
mon = 0;
VALUE s = rb_check_string_type(arg); VALUE s = rb_check_string_type(arg);
if (!NIL_P(s) && RSTRING_LEN(s) > 0) { if (!NIL_P(s) && RSTRING_LEN(s) > 0) {
mon = 0; arg = s;
for (i=0; i<12; i++) { for (i=0; i<12; i++) {
if (RSTRING_LEN(s) == 3 && if (RSTRING_LEN(s) == 3 &&
STRNCASECMP(months[i], RSTRING_PTR(s), 3) == 0) { STRNCASECMP(months[i], RSTRING_PTR(s), 3) == 0) {
@ -2799,15 +2800,8 @@ month_arg(VALUE arg)
break; break;
} }
} }
if (mon == 0) {
char c = RSTRING_PTR(s)[0];
if ('0' <= c && c <= '9') {
mon = obj2ubits(s, 4);
}
}
} }
else { if (mon == 0) {
mon = obj2ubits(arg, 4); mon = obj2ubits(arg, 4);
} }
return mon; return mon;