avcfg: Use av_strdup() instead of g_strdup() for strings owned by ffmpeg
authorSebastian Dröge <sebastian@centricular.com>
Mon, 25 Apr 2016 15:39:54 +0000 (18:39 +0300)
committerSebastian Dröge <sebastian@centricular.com>
Mon, 25 Apr 2016 15:39:54 +0000 (18:39 +0300)
It has its own allocator that depending on the configuration is incompatible
with GLib's and just causes a segmentation fault. Like on Windows.

https://bugzilla.gnome.org/show_bug.cgi?id=760266

ext/libav/gstavcfg.c

index a361994..a1863fd 100644 (file)
@@ -1030,7 +1030,7 @@ gst_ffmpeg_cfg_fill_context (GstFFMpegVidEnc * ffmpegenc,
         /* make a copy for ffmpeg, it will likely free only some,
          * but in any case safer than a potential double free */
         G_STRUCT_MEMBER (gchar *, context, context_offset) =
-            g_strdup (G_STRUCT_MEMBER (gchar *, ffmpegenc, qdata->offset));
+            av_strdup (G_STRUCT_MEMBER (gchar *, ffmpegenc, qdata->offset));
       } else {
         /* memcpy a bit heavy for a small copy,
          * but hardly part of 'inner loop' */