[FFmpeg-devel] More ALS buffer overflows
Måns Rullgård
mans
Fri Feb 19 03:33:59 CET 2010
Reimar D?ffinger <Reimar.Doeffinger at gmx.de> writes:
> That patch is not at all a solution for this issue, at best it will hide it,
> while leaving the possibly exploitable code in.
> The issue here is that the following code snipped has either wrong or missing
> boundary checks:
> current_res = bd->raw_samples + start;
>
> for (sb = 0; sb < sub_blocks; sb++, start = 0)
> for (; start < sb_length; start++)
> *current_res++ = decode_rice(gb, s[sb]);
The patch fixes a real bug regardless of range checks.
--
M?ns Rullg?rd
mans at mansr.com
More information about the ffmpeg-devel
mailing list