On Tue, 2012-10-30 at 08:47 +0530, Arun Raghavan wrote: > 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. Are you applying the patches when reviewing? If so, I think it's ok to push them too if they look good, if the git checkout is ready for pushing anyway. I'm just thinking that there is one redundant step with the "go ahead and push this" process :) -- Tanu
