[Speex-dev] Bug from revision 9316: SB_ENC_STACK misdefined

Thorvald Natvig speex at natvig.com
Mon Jun 6 10:50:50 PDT 2005


Revision 9316 introduced SB_ENC_STACK and friends, but misdefines 
SB_ENC_STACK to be too small, resulting in quite noticable artifacts in 
wideband mode as well as memory corruption and a crash (sooner or later).

The patch below should return SB_ENC_STACK to be equal to what it 
replaced.

Index: sb_celp.c
===================================================================
--- sb_celp.c	(revision 9379)
+++ sb_celp.c	(working copy)
@@ -49,7 +49,7 @@
  /* Default size for the encoder and decoder stack (can be changed at compile time).
     This does not apply when using variable-size arrays or alloca. */
  #ifndef SB_ENC_STACK
-#define SB_ENC_STACK (1000*sizeof(spx_sig_t))
+#define SB_ENC_STACK (10000*sizeof(spx_sig_t))
  #endif

  #ifndef SB_DEC_STACK


More information about the Speex-dev mailing list