[PATCH] resampler: Make sure that there are no overflows when multiplying potentially big numbers.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, 2012-07-09 at 10:07 +0300, Tanu Kaskinen wrote:
> This fixes at least one crash that has been observed. The
> multiplication in trivial_resample() overflowed when
> resampling from 96 kHz to 48 kHz, causing an assertion
> error:
> 
> Assertion 'o_index * fz < pa_memblock_get_length(output->memblock)' failed at pulsecore/resampler.c:1521, function trivial_resample(). Aborting.
> 
> Without the assertion, the memcpy() after the assertion
> would have overwritten some random heap memory.
> ---

Looks good, please go ahead and push this.

-- Arun



[Index of Archives]     [Linux Audio Users]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux