avfvideosrc: Fix wrong default framerate value
authorPiotr Brzeziński <piotr@centricular.com>
Thu, 25 Aug 2022 12:24:25 +0000 (14:24 +0200)
committerGStreamer Marge Bot <gitlab-merge-bot@gstreamer-foundation.org>
Thu, 25 Aug 2022 14:24:57 +0000 (14:24 +0000)
Current default G_MAXINT is not a correct value under any circumstances.
This creates an issue with screen capture, during which we currently do
not get any framerate info causing G_MAXINT to show up, where elements
downstream can possibly misbehave - for example, `vtenc` causes
a kernel panic.
Replace with 30/1 to avoid such scenarios.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2944>

subprojects/gst-plugins-bad/sys/applemedia/avfvideosrc.m

index e357eb0..da205a5 100644 (file)
@@ -1089,7 +1089,7 @@ didOutputSampleBuffer:(CMSampleBufferRef)sampleBuffer
   /* crank up to 11. This is what the presets do, but we don't use the presets
    * in ios >= 7.0 */
   gst_structure_fixate_field_nearest_int (structure, "height", G_MAXINT);
-  gst_structure_fixate_field_nearest_fraction (structure, "framerate", G_MAXINT, 1);
+  gst_structure_fixate_field_nearest_fraction (structure, "framerate", 30, 1);
 
   return gst_caps_fixate (new_caps);
 }