[FFmpeg-cvslog] avcodec/bitstream: Don't check for undefined behaviour after it happened
Andreas Rheinhardt
git at videolan.org
Fri Jul 3 03:10:24 EEST 2020
ffmpeg | branch: release/3.4 | Andreas Rheinhardt <andreas.rheinhardt at gmail.com> | Thu Jun 25 15:10:35 2020 +0200| [660c42ca19d849b19fd60d0622d0ea2cb17249f2] | committer: Andreas Rheinhardt
avcodec/bitstream: Don't check for undefined behaviour after it happened
Reviewed-by: Michael Niedermayer <michael at niedermayer.cc>
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
(cherry picked from commit 5e196dac22cc510db104922f99626a03b453ef4a)
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=660c42ca19d849b19fd60d0622d0ea2cb17249f2
---
libavcodec/bitstream.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavcodec/bitstream.c b/libavcodec/bitstream.c
index 32f3055a6f..de3acf0838 100644
--- a/libavcodec/bitstream.c
+++ b/libavcodec/bitstream.c
@@ -162,9 +162,9 @@ static int build_table(VLC *vlc, int table_nb_bits, int nb_codes,
uint32_t code;
volatile VLC_TYPE (* volatile table)[2]; // the double volatile is needed to prevent an internal compiler error in gcc 4.2
- table_size = 1 << table_nb_bits;
if (table_nb_bits > 30)
return -1;
+ table_size = 1 << table_nb_bits;
table_index = alloc_table(vlc, table_size, flags & INIT_VLC_USE_NEW_STATIC);
ff_dlog(NULL, "new table index=%d size=%d\n", table_index, table_size);
if (table_index < 0)
More information about the ffmpeg-cvslog
mailing list