diff mbox

[FFmpeg-devel] avfilter/avf_avectorscope: Assert that format is valid

Message ID 20170420175859.10938-1-michael@niedermayer.cc
State Accepted
Commit 362f6c91e466b2114d0827c6d58e26e121ed11e8
Headers show

Commit Message

Michael Niedermayer April 20, 2017, 5:58 p.m. UTC 
This should help coverity realize that src[] is inited

Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
---
 libavfilter/avf_avectorscope.c | 2 ++
 1 file changed, 2 insertions(+)

Comments

Paul B Mahol April 20, 2017, 7:33 p.m. UTC | #1 
On 4/20/17, Michael Niedermayer <michael@niedermayer.cc> wrote:
> This should help coverity realize that src[] is inited
>
> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
> ---
>  libavfilter/avf_avectorscope.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/libavfilter/avf_avectorscope.c b/libavfilter/avf_avectorscope.c
> index 3063283efb..c6e278d30d 100644
> --- a/libavfilter/avf_avectorscope.c
> +++ b/libavfilter/avf_avectorscope.c
> @@ -267,6 +267,8 @@ static int filter_frame(AVFilterLink *inlink, AVFrame
> *insamples)
>              src[0] = samplesf[0];
>              src[1] = samplesf[1];
>              break;
> +        default:
> +            av_assert2(0);
>          }
>
>          switch (s->scale) {
> --
> 2.11.0
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>

ok
diff mbox

Patch

diff --git a/libavfilter/avf_avectorscope.c b/libavfilter/avf_avectorscope.c
index 3063283efb..c6e278d30d 100644
--- a/libavfilter/avf_avectorscope.c
+++ b/libavfilter/avf_avectorscope.c
@@ -267,6 +267,8 @@  static int filter_frame(AVFilterLink *inlink, AVFrame *insamples)
             src[0] = samplesf[0];
             src[1] = samplesf[1];
             break;
+        default:
+            av_assert2(0);
         }
 
         switch (s->scale) {