timeline: Fix wrong usage of scale_int
authorThibault Saunier <tsaunier@igalia.com>
Thu, 16 Apr 2020 16:53:00 +0000 (12:53 -0400)
committerThibault Saunier <tsaunier@igalia.com>
Thu, 16 Apr 2020 16:56:10 +0000 (12:56 -0400)
We are multiplying the framerate by GST_SECOND and thus have no
guarantee that it won't overflow.

ges/ges-clip-asset.c
ges/ges-timeline.c

index deb279e..e85e67e 100644 (file)
@@ -230,6 +230,5 @@ ges_clip_asset_get_frame_time (GESClipAsset * self, GESFrameNumber frame_number)
   if (!ges_clip_asset_get_natural_framerate (self, &fps_n, &fps_d))
     return GST_CLOCK_TIME_NONE;
 
-  return gst_util_uint64_scale_int_ceil (frame_number, fps_d * GST_SECOND,
-      fps_n);
+  return gst_util_uint64_scale_ceil (frame_number, fps_d * GST_SECOND, fps_n);
 }
index d8dd8e5..4963fef 100644 (file)
@@ -3109,8 +3109,7 @@ ges_timeline_get_frame_time (GESTimeline * self, GESFrameNumber frame_number)
 
   timeline_get_framerate (self, &fps_n, &fps_d);
 
-  return gst_util_uint64_scale_int_ceil (frame_number,
-      fps_d * GST_SECOND, fps_n);
+  return gst_util_uint64_scale_ceil (frame_number, fps_d * GST_SECOND, fps_n);
 }
 
 /**