Merge pull request #1909 from aeroc7/ub_fixes

Fix undefined behavior: return timestamp of 0 if 'time' is NaN
2022-05-03 15:52:33 -07:00
parent 0e490b94e5 4eba69c305
commit b00609111e
1 changed files with 5 additions and 1 deletions
@@ -287,7 +287,11 @@ int64_t Timecode::time_to_timestamp(const rational &time, const rational &timeba

 int64_t Timecode::time_to_timestamp(const double &time, const rational &timebase, Rounding floor)
 {
-  double d = time * timebase.flipped().toDouble();
+  const double d = time * timebase.flipped().toDouble();
+
+  if (std::isnan(d)) {
+    return 0;
+  }

  switch (floor) {
  case kRound: