comparison dsp/chromagram/ConstantQ.cpp @ 38:3dff6e3e2121

* Ouch!  We overrun the FFTRe/Im vectors with the very first dereference
  I assume the kernel matrix column values are supposed to index the FFT
  bins from the end, not from the end plus one

     {
 	const unsigned row = cqbin[i];
 	const unsigned col = fftbin[i];
 	const double & r1  = real[i];
 	const double & i1  = imag[i];
-	const double & r2  = fftdata[ (2*m_FFTLength) - 2*col];
-	const double & i2  = fftdata[ (2*m_FFTLength) - 2*col+1];
+	const double & r2  = fftdata[ (2*m_FFTLength) - 2*col - 2 ];
+	const double & i2  = fftdata[ (2*m_FFTLength) - 2*col - 2 + 1 ];
 	// add the multiplication
 	m_CQdata[ 2*row  ] += (r1*r2 - i1*i2);
 	m_CQdata[ 2*row+1] += (r1*i2 + i1*r2);
     }
 

     {
 	const unsigned row = cqbin[i];
 	const unsigned col = fftbin[i];
 	const double & r1  = real[i];
 	const double & i1  = imag[i];
-	const double & r2  = FFTRe[ m_FFTLength - col];
-	const double & i2  = FFTIm[ m_FFTLength - col];
+	const double & r2  = FFTRe[ m_FFTLength - col - 1 ];
+	const double & i2  = FFTIm[ m_FFTLength - col - 1 ];
 	// add the multiplication
 	CQRe[ row ] += (r1*r2 - i1*i2);
 	CQIm[ row ] += (r1*i2 + i1*r2);
     }
 }