david/librubberband
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Ensure we don't overrun the mid-side preprocessing buffer

Browse Source
This commit is contained in:
Chris Cannam
2022-01-25 13:29:23 +00:00
parent c7cfc2a1b9
commit 347fc3e00d
1 changed files with 9 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
src/StretcherProcess.cpp
View File

@@ -197,6 +197,15 @@ RubberBandStretcher::Impl::consumeChannel(size_t c,
if (samples == 0) return 0; if (samples == 0) return 0;
} }
if (useMidSide) {
// Our pre-resample buffer size is also constrained by a
// buffer (cd.ms) of the same size as cd.inbuf
size_t ibsz = cd.inbuf->getSize();
if (ibsz < samples) {
samples = ibsz;
}
}
size_t reqSize = int(ceil(samples / m_pitchScale)); size_t reqSize = int(ceil(samples / m_pitchScale));
if (reqSize > cd.resamplebufSize) { if (reqSize > cd.resamplebufSize) {
cerr << "WARNING: RubberBandStretcher::Impl::consumeChannel: resizing resampler buffer from " cerr << "WARNING: RubberBandStretcher::Impl::consumeChannel: resizing resampler buffer from "