ext/ffmpeg/gstffmpegenc.c: Don't crash if avcodec_open failed (#301717).
authorRonald S. Bultje <rbultje@ronald.bitfreak.net>
Sat, 23 Apr 2005 16:07:34 +0000 (16:07 +0000)
committerRonald S. Bultje <rbultje@ronald.bitfreak.net>
Sat, 23 Apr 2005 16:07:34 +0000 (16:07 +0000)
Original commit message from CVS:
Reviewed by:  Ronald S. Bultje  <rbultje@ronald.bitfreak.net>
* ext/ffmpeg/gstffmpegenc.c: (gst_ffmpegenc_getcaps):
Don't crash if avcodec_open failed (#301717).

ChangeLog
ext/ffmpeg/gstffmpegenc.c

index 9c1d9f3..f588aaa 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2005-04-23  Luca Ognibene  <luogni@tin.it>
+
+       Reviewed by:  Ronald S. Bultje  <rbultje@ronald.bitfreak.net>
+
+       * ext/ffmpeg/gstffmpegenc.c: (gst_ffmpegenc_getcaps):
+         Don't crash if avcodec_open failed (#301717).
+
 2005-04-21  Ronald S. Bultje  <rbultje@ronald.bitfreak.net>
 
        * ext/ffmpeg/gstffmpegdemux.c: (gst_ffmpegdemux_register):
index 4e7c2b0..cc815f1 100644 (file)
@@ -331,9 +331,8 @@ gst_ffmpegenc_getcaps (GstPad * pad)
         caps = gst_caps_new_empty ();
       gst_caps_append (caps,
           gst_ffmpeg_codectype_to_caps (oclass->in_plugin->type, ctx));
+      avcodec_close (ctx);
     }
-    /* FIXME: ffmpeg likes to crash on this */
-    avcodec_close (ctx);
   }
   av_free (ctx);