diff --git a/src/lib.rs b/src/lib.rs index c05597d..adff6bf 100644 --- a/src/lib.rs +++ b/src/lib.rs @@ -74,13 +74,12 @@ impl From for ParseInternalError { pub enum ParseError { AmbiguousWeekday, InternalError(ParseInternalError), - InvalidDay, InvalidMonth, UnrecognizedToken(String), InvalidParseResult(ParsingResult), AmPmWithoutHour, - InvalidHour, TimezoneUnsupported, + ImpossibleTimestamp(&'static str), } impl From for ParseError { @@ -774,7 +773,7 @@ impl Parser { if fuzzy { Ok(false) } else { - Err(ParseError::InvalidHour) + Err(ParseError::ImpossibleTimestamp("Invalid hour")) } } else { Ok(false) @@ -806,13 +805,24 @@ impl Parser { let d = d + d_offset; - let t = NaiveTime::from_hms_micro( - res.hour.unwrap_or(default.hour() as i32) as u32, - res.minute.unwrap_or(default.minute() as i32) as u32, - res.second.unwrap_or(default.second() as i32) as u32, - res.microsecond - .unwrap_or(default.timestamp_subsec_micros() as i32) as u32, - ); + let hour = res.hour.unwrap_or(default.hour() as i32) as u32; + let minute = res.minute.unwrap_or(default.minute() as i32) as u32; + let second = res.second.unwrap_or(default.second() as i32) as u32; + let microsecond = res.microsecond + .unwrap_or(default.timestamp_subsec_micros() as i32) as u32; + let t = NaiveTime::from_hms_micro_opt(hour, minute, second, microsecond).ok_or_else(|| { + if hour >= 24 { + ParseError::ImpossibleTimestamp("Invalid hour") + } else if minute >= 60 { + ParseError::ImpossibleTimestamp("Invalid minute") + } else if second >= 60 { + ParseError::ImpossibleTimestamp("Invalid second") + } else if microsecond >= 2_000_000 { + ParseError::ImpossibleTimestamp("Invalid microsecond") + } else { + unreachable!(); + } + })?; Ok(NaiveDateTime::new(d, t)) } diff --git a/src/tests.rs b/src/tests.rs index 16e5bc0..5f0d842 100644 --- a/src/tests.rs +++ b/src/tests.rs @@ -14,4 +14,6 @@ fn test_fuzz() { let mut p = Parser::default(); let res = p.parse("\x0D\x31", None, None, false, false, Some(&default), false, HashMap::new()).unwrap(); assert_eq!(res.0, default); -} \ No newline at end of file + + assert_eq!(parse("\x2D\x2D\x32\x31\x38\x6D"), Err(ParseError::ImpossibleTimestamp("Invalid minute"))); +}