[FFmpeg-devel] [PATCH 2/7] avcodec/speedhq: Don't pretend reading DC can fail

Andreas Rheinhardt andreas.rheinhardt at gmail.com
Mon Oct 12 11:12:12 EEST 2020


It can't, because the tables used don't have any loose ends. This also
fixes a bug in the only caller of decode_dc_le(): It didn't check the
return value.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
---
 libavcodec/speedhq.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/libavcodec/speedhq.c b/libavcodec/speedhq.c
index b834b79f28..64758164de 100644
--- a/libavcodec/speedhq.c
+++ b/libavcodec/speedhq.c
@@ -168,10 +168,6 @@ static inline int decode_dc_le(GetBitContext *gb, int component)
     } else {
         code = get_vlc2(gb, ff_dc_chroma_vlc_le.table, DC_VLC_BITS, 2);
     }
-    if (code < 0) {
-        av_log(NULL, AV_LOG_ERROR, "invalid dc code at\n");
-        return 0xffff;
-    }
     if (!code) {
         diff = 0;
     } else {
-- 
2.25.1



More information about the ffmpeg-devel mailing list