[FFmpeg-cvslog] lavfi/buffersrc: do not dereference NULL frame.

Nicolas George git at videolan.org
Sun Mar 10 19:04:54 CET 2013


ffmpeg | branch: master | Nicolas George <nicolas.george at normalesup.org> | Sun Mar 10 16:28:33 2013 +0100| [b71db3f38eafeb05e068804e002984b0d232e5f6] | committer: Nicolas George

lavfi/buffersrc: do not dereference NULL frame.

frame = NULL is used to indicate EOF.

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

 libavfilter/buffersrc.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/libavfilter/buffersrc.c b/libavfilter/buffersrc.c
index 185018d..be860aa 100644
--- a/libavfilter/buffersrc.c
+++ b/libavfilter/buffersrc.c
@@ -93,9 +93,9 @@ int av_buffersrc_add_frame_flags(AVFilterContext *ctx, AVFrame *frame, int flags
 {
     AVFrame *copy = NULL;
     int ret = 0;
-    int64_t layout = frame->channel_layout;
 
-    if (layout && av_get_channel_layout_nb_channels(layout) != av_frame_get_channels(frame)) {
+    if (frame && frame->channel_layout &&
+        av_get_channel_layout_nb_channels(frame->channel_layout) != av_frame_get_channels(frame)) {
         av_log(0, AV_LOG_ERROR, "Layout indicates a different number of channels than actually present\n");
         return AVERROR(EINVAL);
     }



More information about the ffmpeg-cvslog mailing list