From: Arnout Vandecappelle <arnout at mind dot be>
Date: Mon, 23 Feb 2009 11:14:23 +0000 (+0100)
Subject: Don't do crazy things with 0/1 framerates
X-Git-Tag: RELEASE-0.10.15~411^2~29
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=b9adb5846bb4d663928e441cf90e1efb75f36630;p=platform%2Fupstream%2Fgst-plugins-good.git

Don't do crazy things with 0/1 framerates

We use 0/1 framerates to mark variable framerates and matroskamux should not try
to calculate a frame duration for it.
Fixes #571294.
---

diff --git a/gst/matroska/matroska-mux.c b/gst/matroska/matroska-mux.c
index 746e3d194..1b958d69e 100644
--- a/gst/matroska/matroska-mux.c
+++ b/gst/matroska/matroska-mux.c
@@ -650,7 +650,8 @@ gst_matroska_mux_video_pad_setcaps (GstPad * pad, GstCaps * caps)
   gst_structure_get_int (structure, "height", &height);
   videocontext->pixel_width = width;
   videocontext->pixel_height = height;
-  if (gst_structure_get_fraction (structure, "framerate", &fps_n, &fps_d)) {
+  if (gst_structure_get_fraction (structure, "framerate", &fps_n, &fps_d)
+      && fps_n > 0) {
     context->default_duration =
         gst_util_uint64_scale_int (GST_SECOND, fps_d, fps_n);
     GST_LOG_OBJECT (pad, "default duration = %" GST_TIME_FORMAT,