diff options
author | Andreas Cadhalpun <andreas.cadhalpun@googlemail.com> | 2015-05-11 20:07:49 +0200 |
---|---|---|
committer | Michael Niedermayer <michaelni@gmx.at> | 2015-05-21 20:43:38 +0200 |
commit | 596b9d6ed88fc9a04f4e016380eb209f3f7ecd72 (patch) | |
tree | a947619308022f0ca9629ddbb8a926c51dbbf018 /libavformat/realtextdec.c | |
parent | 699357dd4ea492a83d84baac287cfd363974f8b2 (diff) | |
download | ffmpeg-596b9d6ed88fc9a04f4e016380eb209f3f7ecd72.tar.gz |
id3v2: catch avio_read errors in check_tag
Since len is an unsigned int, the comparison is currently treated as
unsigned and thus ignores all errors from avio_read.
Thus cast len to int, which is unproblematic, because at that point len
is between 0 and 4.
This fixes 'Conditional jump or move depends on uninitialised value'
valgrind warnings in is_tag.
Signed-off-by: Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>
Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
(cherry picked from commit 0382c94f13b4b20456b7259e90b170dc020419b8)
Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
Diffstat (limited to 'libavformat/realtextdec.c')
0 files changed, 0 insertions, 0 deletions