diff options
| author | Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com> | 2015-04-28 22:37:19 +0200 |
|---|---|---|
| committer | Michael Niedermayer <michaelni@gmx.at> | 2015-06-01 23:25:21 +0200 |
| commit | 35a0d4801f26a3f695ac5bd75f90ebea735ef86b (patch) | |
| tree | df3860480cb3c79b53d26ab01a062cd2e0a7654b /libavcodec/vorbis_data.c | |
| parent | e0ef1b8c0b21122850458d1c2c229d8bd852a74d (diff) | |
| download | ffmpeg-35a0d4801f26a3f695ac5bd75f90ebea735ef86b.tar.gz | |
nutdec: fix illegal count check in decode_main_header
The existing check has two problems:
1) i + count can overflow, so that the check '< 256' returns true.
2) In the (i == 'N') case occurs a j-- so that the loop runs once more.
This can trigger the assertion 'nut->header_len[0] == 0' or cause
segmentation faults or infinite hangs.
Signed-off-by: Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>
Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
(cherry picked from commit 7c24ca1bda2d4df1dc9b2b982941be532d60da21)
Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
Diffstat (limited to 'libavcodec/vorbis_data.c')
0 files changed, 0 insertions, 0 deletions