Unfortunately we are missing bool Time::TryFromTime(hh, mm, ss, us, off, result), so this will not fail if the time is invalid. I think we need to move this duplicated code into time.cpp and add a call to IsValidTime at the end. Among other things I'm seeing that the parsing code for timestamps does not accept 24:00:00, which is valid. This may be more than you bargained for, so unless you want to wade in deeper, this is fine for now.

Originally posted by @hawkfish in #11516 (comment)