ivfdec: tolerate invalid framerates (2)
authorJames Zern <jzern@google.com>
Sat, 4 Jun 2016 01:04:54 +0000 (18:04 -0700)
committerJames Zern <jzern@google.com>
Sat, 4 Jun 2016 01:04:54 +0000 (18:04 -0700)
add an upper bound to the framerate denominator above which 30fps will
be reported; fixes warning in corrupt / fuzzed files

Change-Id: I46a6a6f34ab756535cd009fe12273d83dcc1e9f1

ivfdec.c

index 6d1d679..7fc25a0 100644 (file)
--- a/ivfdec.c
+++ b/ivfdec.c
@@ -23,7 +23,7 @@ static void fix_framerate(int *num, int *den) {
   // we can guess the framerate using only the timebase in this
   // case. Other files would require reading ahead to guess the
   // timebase, like we do for webm.
-  if (*den > 0 && *num > 0 && *num < 1000) {
+  if (*den > 0 && *den < 1000000000 && *num > 0 && *num < 1000) {
     // Correct for the factor of 2 applied to the timebase in the encoder.
     if (*num & 1)
       *den *= 2;