diff mbox series

[FFmpeg-devel] avcodec/clearvideo: Don't check for errors for complete VLC

Message ID 20201024144036.92901-1-andreas.rheinhardt@gmail.com
State Accepted
Commit 1d658e65ab60ff0a542d875480474fb16e3413fd
Headers show
Series [FFmpeg-devel] avcodec/clearvideo: Don't check for errors for complete VLC | expand

Checks

Context Check Description
andriy/x86_make success Make finished
andriy/x86_make_fate success Make fate finished
andriy/PPC64_make success Make finished
andriy/PPC64_make_fate success Make fate finished

Commit Message

Andreas Rheinhardt Oct. 24, 2020, 2:40 p.m. UTC 
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
---
 libavcodec/clearvideo.c | 2 --
 1 file changed, 2 deletions(-)

Comments

Andreas Rheinhardt Oct. 28, 2020, 5:56 a.m. UTC | #1 
Andreas Rheinhardt:
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
> ---
>  libavcodec/clearvideo.c | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/libavcodec/clearvideo.c b/libavcodec/clearvideo.c
> index 65bf140401..73c0367f71 100644
> --- a/libavcodec/clearvideo.c
> +++ b/libavcodec/clearvideo.c
> @@ -88,8 +88,6 @@ static inline int decode_block(CLVContext *ctx, int16_t *blk, int has_ac,
>  
>      memset(blk, 0, sizeof(*blk) * 64);
>      blk[0] = get_vlc2(gb, ctx->dc_vlc.table, 9, 3);
> -    if (blk[0] < 0)
> -        return AVERROR_INVALIDDATA;
>      blk[0] -= 63;
>  
>      if (!has_ac)
> 
Will apply this later today unless there are objections.

- Andreas
diff mbox series

Patch

diff --git a/libavcodec/clearvideo.c b/libavcodec/clearvideo.c
index 65bf140401..73c0367f71 100644
--- a/libavcodec/clearvideo.c
+++ b/libavcodec/clearvideo.c
@@ -88,8 +88,6 @@  static inline int decode_block(CLVContext *ctx, int16_t *blk, int has_ac,
 
     memset(blk, 0, sizeof(*blk) * 64);
     blk[0] = get_vlc2(gb, ctx->dc_vlc.table, 9, 3);
-    if (blk[0] < 0)
-        return AVERROR_INVALIDDATA;
     blk[0] -= 63;
 
     if (!has_ac)