[FFmpeg-cvslog] vf_yadif: Fixed potential access violation on EOF.

Robert Nagy git at videolan.org
Tue May 22 16:06:49 CEST 2012


ffmpeg | branch: master | Robert Nagy <ronag89 at gmail.com> | Tue May 22 00:14:48 2012 +0200| [965b8072781b439b36835a3df345a4c68e0c211f] | committer: Michael Niedermayer

vf_yadif: Fixed potential access violation on EOF.

The correct invariant is (cur != NULL -> next != NULL). Not the other way around.

Signed-off-by: Michael Niedermayer <michaelni at gmx.at>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=965b8072781b439b36835a3df345a4c68e0c211f
---

 libavfilter/vf_yadif.c |    5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/libavfilter/vf_yadif.c b/libavfilter/vf_yadif.c
index fb4862d..d7fa98a 100644
--- a/libavfilter/vf_yadif.c
+++ b/libavfilter/vf_yadif.c
@@ -17,6 +17,7 @@
  * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
  */
 
+#include "libavutil/avassert.h"
 #include "libavutil/cpu.h"
 #include "libavutil/common.h"
 #include "libavutil/pixdesc.h"
@@ -238,6 +239,8 @@ static void start_frame(AVFilterLink *link, AVFilterBufferRef *picref)
     AVFilterContext *ctx = link->dst;
     YADIFContext *yadif = ctx->priv;
 
+    av_assert0(picref);
+
     if (yadif->frame_pending)
         return_frame(ctx, 1);
 
@@ -308,7 +311,7 @@ static int request_frame(AVFilterLink *link)
 
         ret  = avfilter_request_frame(link->src->inputs[0]);
 
-        if (ret == AVERROR_EOF && yadif->next) {
+        if (ret == AVERROR_EOF && yadif->cur) {
             AVFilterBufferRef *next = avfilter_ref_buffer(yadif->next, AV_PERM_READ);
             next->pts = yadif->next->pts * 2 - yadif->cur->pts;
 



More information about the ffmpeg-cvslog mailing list